[libvirt] [PATCH] esx: Fix a potential crash
Matthias Bolte
matthias.bolte at googlemail.com
Tue Jul 12 15:12:10 UTC 2011
2011/7/12 Osier Yang <jyang at redhat.com>:
> 于 2011年07月12日 17:50, Matthias Bolte 写道:
>>
>> 2011/7/12 Osier Yang<jyang at redhat.com>:
>>>
>>> 于 2011年07月12日 16:58, Daniel Veillard 写道:
>>>>
>>>> On Tue, Jul 12, 2011 at 05:05:16PM +0800, Osier Yang wrote:
>>>>>
>>>>> Initialize ptr data.datastorePathWithoutFileName as NULL, otherwise
>>>>> it might cause crash when trying to free it in cleanup.
>>>>>
>>>>> ---
>>>>> src/esx/esx_driver.c | 1 +
>>>>> 1 files changed, 1 insertions(+), 0 deletions(-)
>>>>>
>>>>> diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c
>>>>> index ddb8c23..ae614a6 100644
>>>>> --- a/src/esx/esx_driver.c
>>>>> +++ b/src/esx/esx_driver.c
>>>>> @@ -2698,6 +2698,7 @@ esxDomainGetXMLDesc(virDomainPtr domain, unsigned
>>>>> int flags)
>>>>> char *vmx = NULL;
>>>>> virVMXContext ctx;
>>>>> esxVMX_Data data;
>>>>> + data.datastorePathWithoutFileName = NULL;
>>>>> virDomainDefPtr def = NULL;
>>>>> char *xml = NULL;
>>>>
>>>> ACK,
>>>>
>>>> Daniel
>>>>
>>> Thanks, applied
>>>
>>> Osier
>>
>> Yes, this fixes the problem.
>>
>> I'd like to propose this more general approach here.
>>
> Yes, this is more general, and fix all the similiar problems, ACK, please
> revert my
> commit if need when you push. Thanks
>
> Regards
> Osier
>
Pushed now, thanks.
--
Matthias Bolte
http://photron.blogspot.com
More information about the libvir-list
mailing list