[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH trivial v2] block.c: Add return value for bdrv
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-trivial] [PATCH trivial v2] block.c: Add return value for bdrv_append_temp_snapshot() to avoid incorrect failure processing issue |
Date: |
Fri, 27 Jun 2014 19:21:57 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
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>
Thanks, applied to the block branch.
Kevin