Commit 622d2419 authored by Paolo Bonzini's avatar Paolo Bonzini Committed by Kevin Wolf
Browse files

use QSIMPLEQ_FOREACH_SAFE when freeing list elements 



Signed-off-by: default avatarPaolo Bonzini <pbonzini@redhat.com>
Signed-off-by: default avatarKevin Wolf <kwolf@redhat.com>
parent b8c6f29e

blockdev.c

+2 −2
Original line number Diff line number Diff line
@@ -736,7 +736,7 @@ void qmp_blockdev_group_snapshot_sync(SnapshotDevList *dev_list, 
    int ret = 0;

    SnapshotDevList *dev_entry = dev_list;

    SnapshotDev *dev_info = NULL;

    BlkGroupSnapshotStates *states;

    BlkGroupSnapshotStates *states, *next;

    BlockDriver *proto_drv;

    BlockDriver *drv;

    int flags;
@@ -842,7 +842,7 @@ delete_and_fail: 
        }

    }

exit:

    QSIMPLEQ_FOREACH(states, &snap_bdrv_states, entry) {

    QSIMPLEQ_FOREACH_SAFE(states, &snap_bdrv_states, entry, next) {

        g_free(states);

    }

    return;