On 07/25/2017 10:02 AM, Philippe Mathieu-Daudé wrote: > Maybe worth adding "Coverity: CID 1377700" At this point, the PULL request is already posted, so it may be too late. But it's not the end of the world if we miss the extra comment. > > On 07/14/2017 01:00 PM, Max Reitz wrote: >> On 2017-07-14 14:33, Vladimir Sementsov-Ogievskiy wrote: >>> Fix possible crash on error path in >>> qcow2_remove_persistent_dirty_bitmap. Although bitmap_free was added in >>> 88ddffae8fc the bug was introduced later in commit 469c71edc72 (when >>> qcow2_remove_persistent_dirty_bitmap was added). >>> >>> Signed-off-by: Vladimir Sementsov-Ogievskiy >>> Reviewed-by: Eric Blake >>> --- >>> block/qcow2-bitmap.c | 4 ++++ >>> 1 file changed, 4 insertions(+) >> >> Thanks, applied to my block branch: >> >> https://github.com/XanClic/qemu/commits/block >> >> Max >> > -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org