[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH] block/qapi: Clear err for further
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH] block/qapi: Clear err for further error |
Date: |
Mon, 11 Jan 2016 16:41:58 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Max Reitz <address@hidden> writes:
> On 22.12.2015 15:39, John Snow wrote:
>>
>>
>> On 12/22/2015 05:11 AM, Fam Zheng wrote:
>>> Since a5002d5 (block/qapi: allow best-effort query) we don't return at
>>> this error, however err must be cleared before passing to
>>> bdrv_query_snapshot_info_list below, as required by error API.
>>>
>>> Signed-off-by: Fam Zheng <address@hidden>
>>> ---
>>> block/qapi.c | 1 +
>>> 1 file changed, 1 insertion(+)
>>>
>>> diff --git a/block/qapi.c b/block/qapi.c
>>> index fecac25..58d3975 100644
>>> --- a/block/qapi.c
>>> +++ b/block/qapi.c
>>> @@ -250,6 +250,7 @@ void bdrv_query_image_info(BlockDriverState *bs,
>>> g_free(backing_filename2);
>>> backing_filename2 = NULL;
>>> error_free(err);
>>> + err = NULL;
>>> }
>>>
>>> /* Always report the full_backing_filename if present, even if
>>> it's the
>>>
>>
>> Sorry Markus, I've failed you.
There, for the grace of God, go I.
>> Reviewed-by: John Snow <address@hidden>
>
> As did I by giving an R-b and applying it.
>
> Let us hereby atone for our sins:
>
> https://github.com/XanClic/qemu/commits/block
>
>
> I hope the spirit of christmas will make Markus forgive us.
Happy new year!
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-block] [Qemu-devel] [PATCH] block/qapi: Clear err for further error,
Markus Armbruster <=