[libvirt] [PATCH] esx: Fix potential memory leak in esxVI_BuildFullTraversalSpecItem
Matthias Bolte
matthias.bolte at googlemail.com
Tue Mar 23 00:34:14 UTC 2010
2010/3/22 Daniel P. Berrange <berrange at redhat.com>:
> On Mon, Mar 22, 2010 at 01:55:45AM +0100, Matthias Bolte wrote:
>> If esxVI_String_DeepCopyValue or esxVI_SelectionSpec_AppendToList fail
>> then selectionSpec would leak. Add a free call in the failure path to
>> fix the leak.
>> ---
>> src/esx/esx_vi.c | 4 ++--
>> 1 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/esx/esx_vi.c b/src/esx/esx_vi.c
>> index aa46532..63ddaa4 100644
>> --- a/src/esx/esx_vi.c
>> +++ b/src/esx/esx_vi.c
>> @@ -1163,8 +1163,6 @@ esxVI_BuildFullTraversalSpecItem(esxVI_SelectionSpec **fullTraversalSpecList,
>> currentSelectSetName = selectSetNames;
>>
>> while (currentSelectSetName != NULL && *currentSelectSetName != '\0') {
>> - selectionSpec = NULL;
>> -
>> if (esxVI_SelectionSpec_Alloc(&selectionSpec) < 0 ||
>> esxVI_String_DeepCopyValue(&selectionSpec->name,
>> currentSelectSetName) < 0 ||
>> @@ -1173,6 +1171,7 @@ esxVI_BuildFullTraversalSpecItem(esxVI_SelectionSpec **fullTraversalSpecList,
>> goto failure;
>> }
>>
>> + selectionSpec = NULL;
>> currentSelectSetName += strlen(currentSelectSetName) + 1;
>> }
>> }
>> @@ -1186,6 +1185,7 @@ esxVI_BuildFullTraversalSpecItem(esxVI_SelectionSpec **fullTraversalSpecList,
>>
>> failure:
>> esxVI_TraversalSpec_Free(&traversalSpec);
>> + esxVI_SelectionSpec_Free(&selectionSpec);
>>
>> return -1;
>> }
>
> ACK
>
>
> Daniel
>
Thanks, pushed.
Matthias
More information about the libvir-list
mailing list