[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