Commit b6b1d31f authored by Stefan Hajnoczi's avatar Stefan Hajnoczi
Browse files

vmdk: fix buf leak in vmdk_parse_extents() 



vmdk_open_sparse() does not take ownership of buf so the caller always
needs to free it.

Signed-off-by: default avatarStefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: default avatarMax Reitz <mreitz@redhat.com>
Reviewed-by: default avatarFam Zheng <famz@redhat.com>
parent ff74f33c

block/vmdk.c

+1 −1
Original line number Diff line number Diff line
@@ -846,8 +846,8 @@ static int vmdk_parse_extents(const char *desc, BlockDriverState *bs, 
            } else {

                ret = vmdk_open_sparse(bs, extent_file, bs->open_flags, buf, errp);

            }

            if (ret) {

            g_free(buf);

            if (ret) {

                bdrv_unref(extent_file);

                return ret;

            }