On 07/17/13 11:57, Fam Zheng wrote: > s->qcow and s->qcow_filename are allocated but not freed on error. Fix the > possible leaks, remove unnecessary check for bdrv_new(), propagate ret code of > bdrv_create() and also the one of enable_write_target(). > > Signed-off-by: Fam Zheng <f...@redhat.com> > --- > > v2: Fix leak of s->qcow_filename, propagate returen value of > enable_write_target(). [Laszlo] > > --- > block/vvfat.c | 25 ++++++++++++++----------- > 1 file changed, 14 insertions(+), 11 deletions(-)
Looks good to me. Reviewed-by: Laszlo Ersek <ler...@redhat.com>