[libvirt] [PATCH 1/3] virterror: Make SetError work if no previous error was set

Cole Robinson crobinso at redhat.com
Thu Feb 18 14:42:41 UTC 2010


On 02/18/2010 07:45 AM, Daniel P. Berrange wrote:
> On Tue, Feb 16, 2010 at 02:37:00PM -0500, Cole Robinson wrote:
>> virGetLastError returns NULL if no error has been set, not on
>> allocation error like virSetError assumed. Use virLastErrorObject
>> instead. This fixes virSetError when no error is currently stored.
>>
>> Signed-off-by: Cole Robinson <crobinso at redhat.com>
>> ---
>>  src/util/virterror.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/util/virterror.c b/src/util/virterror.c
>> index bbf5021..cbd0ca8 100644
>> --- a/src/util/virterror.c
>> +++ b/src/util/virterror.c
>> @@ -304,7 +304,7 @@ int
>>  virSetError(virErrorPtr newerr)
>>  {
>>      virErrorPtr err;
>> -    err = virGetLastError();
>> +    err = virLastErrorObject();
>>      if (!err)
>>          return -1;
>>  
>> -- 
> 
> ACK
> 
> Daniel

Thanks, I've pushed these 3 now.

- Cole




More information about the libvir-list mailing list