[libvirt] Free name in xenDaemonLookupByID(), if virGetDomain() fails

Matthias Bolte matthias.bolte at googlemail.com
Tue Sep 8 13:04:38 UTC 2009


2009/9/8 Daniel Veillard <veillard at redhat.com>:
> On Tue, Sep 08, 2009 at 02:35:40PM +0200, Matthias Bolte wrote:
>> Subject says it all.
>>
>> Matthias
>
>> diff --git a/src/xend_internal.c b/src/xend_internal.c
>> index 2fa08f1..ed6fcac 100644
>> --- a/src/xend_internal.c
>> +++ b/src/xend_internal.c
>> @@ -3723,7 +3723,7 @@ xenDaemonLookupByID(virConnectPtr conn, int id) {
>>      }
>>
>>      ret = virGetDomain(conn, name, uuid);
>> -    if (ret == NULL) return NULL;
>> +    if (ret == NULL) goto error;
>>
>>      ret->id = id;
>>      VIR_FREE(name);
>
>
>  ACK, applied !
> Thanks for chasing and for the patch mime-type :-)
>
> Daniel
>

For some reason the Google Mail web client detects patches with .patch
extension as application/mbox now. If I rename it to .diff, it's
correctly detected as text/x-diff. There seems to be no way to
explicitly tell the client which mime-type an attachment should have.

Anyway, here is another patch for a similar leak in xenDaemonLookupByUUID().

Matthias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix_second_xend_name_leak.diff
Type: text/x-diff
Size: 447 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20090908/a52077dd/attachment-0001.bin>


More information about the libvir-list mailing list