[Top][All Lists]

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

Re: [Qemu-trivial] [Qemu-devel] [PATCH trivial v2] block.c: Add return v

From: Stefan Hajnoczi
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH trivial v2] block.c: Add return value for bdrv_append_temp_snapshot() to avoid incorrect failure processing issue
Date: Fri, 27 Jun 2014 13:50:32 +0200
User-agent: Mutt/1.5.23 (2014-03-12)

On Tue, Jun 24, 2014 at 01:01:52PM +0200, Markus Armbruster wrote:
> Kevin Wolf <address@hidden> writes:
> > Am 23.06.2014 um 17:28 hat Chen Gang geschrieben:
> >> When failure occurs, 'ret' need be set, or may return 0 to indicate 
> >> success.
> >> And error_propagate() also need be called only one time within a function.
> >> 
> >> It is abnormal to prevent bdrv_append_temp_snapshot() return value but 
> >> still
> >> set errp when error occurs -- although it contents return value internally.
> >> 
> >> So let bdrv_append_temp_snapshot() internal return value outside, and let
> >> all things normal, then fix the issue too.
> >> 
> >> Signed-off-by: Chen Gang <address@hidden>
> >
> > What does this fix?
> It fixes the return value of bdrv_open() when
> bdrv_append_temp_snapshot() fails.  Before this patch, it returns a
> positive value, which is wrong.  After the patch, it returns the
> negative error code bdrv_append_temp_snapshot() now returns.

Exactly.  I asked for the -errno return because otherwise bdrv_open()
would have no accurate errno.


Attachment: pgpBgCBhaQruA.pgp
Description: PGP signature

reply via email to

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