[libvirt] [PATCH 4/4] qemu: fix a error coverd issue in 2 place
lhuang
lhuang at redhat.com
Fri Feb 27 03:40:47 UTC 2015
On 02/25/2015 10:51 PM, Ján Tomko wrote:
> On Tue, Feb 24, 2015 at 01:52:08PM -0500, John Ferlan wrote:
>>
>> On 02/13/2015 02:17 AM, Luyao Huang wrote:
>>> we already set a more clearly error in networkGetNetworkAddress,
>>> so this error will cover our error in networkGetNetworkAddress.
>>>
>>> Signed-off-by: Luyao Huang <lhuang at redhat.com>
>>> ---
>>> src/qemu/qemu_command.c | 14 ++++----------
>>> 1 file changed, 4 insertions(+), 10 deletions(-)
>>>
>> I didn't check every exit patch from networkGetNetworkAddress, but
>> perhaps these should change to :
>>
>> if (ret < 0) {
>> if (virGetLastError() == NULL)
>> virReportError(VIR_ERR_XML_ERROR,...);
>> goto error;
>> }
>>
>> Just to be sure we don't leave without setting some sort of error.
> No.
>
> All the exit paths should be checked to make sure an error was reported
> in all cases where we return -1.
Okay, i have checked the code and think we will return -1 with an error
setting.
So i will keep the code like this:
if (ret < 0)
goto error;
Thanks for your review
> Jan
Luyao
More information about the libvir-list
mailing list