[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [Qemu-devel] [PULL 13/15] qemu-img: Allow rebase with n

From: Peter Maydell
Subject: Re: [Qemu-block] [Qemu-devel] [PULL 13/15] qemu-img: Allow rebase with no input base
Date: Mon, 20 May 2019 16:33:03 +0100

On Fri, 10 May 2019 at 17:37, Kevin Wolf <address@hidden> wrote:
> From: Max Reitz <address@hidden>
> Currently, without -u, you cannot add a backing file to an image when it
> currently has none:
> $ qemu-img rebase -b base.qcow2 foo.qcow2
> qemu-img: Could not open old backing file '': The 'file' block driver
> requires a file name
> It is really simple to allow this, though (effectively by setting
> old_backing_size to 0), so this patch does just that.
> Signed-off-by: Max Reitz <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> Signed-off-by: Kevin Wolf <address@hidden>

Hi -- Coverity reports some new "resource leak" errors
in the qemu-img.c img_rebase() function:
 * CID 1401415 : Coverity complains that the memory assigned
   to out_real_path gets leaked if we take the immediately
   following "if (local_err) ... goto out" error-exit path.
   This may be another instance of Coverity not understanding
   that the function won't return non-NULL if errp is set.
 * CID 1401416 : leak of the 'options' qdict in this
   error-exit code path. This looks like a real issue (and
   there seem to be other error-exit codepaths earlier in
   the function which leak this too.)

Would somebody like to have a look at these?

-- PMM

reply via email to

[Prev in Thread] Current Thread [Next in Thread]