[libvirt] [PATCH 3/3] parallels: fix possible crash in case of errors in prlsdkLoadDomain

Maxim Nestratov mnestratov at virtuozzo.com
Wed May 20 12:38:18 UTC 2015

20.05.2015 15:32, Daniel P. Berrange пишет:
> On Mon, May 18, 2015 at 05:44:20PM +0300, Maxim Nestratov wrote:
>> Cleanup code in prlsdkLoadDomain doesn't take into account the fact
>> if private domain freeing function is assigned or not. In case it is,
>> we shouldn't call it manually because virDomainObjListRemove calls it.
>> Also, allocated def structure should be freed only if it's not
>> assigned to domain. Otherwise it will be called twice one time by
>> virDomainObjListRemove and the second by prlsdkLoadDomain itself.
>> Signed-off-by: Maxim Nestratov <mnestratov at parallels.com>
>> ---
>>   src/parallels/parallels_sdk.c | 17 ++++++++++++++---
>>   1 file changed, 14 insertions(+), 3 deletions(-)
> ACK, but will let Dmitry sanity check and push this series too.
> Regards,
> Daniel
Ok. Thank you Daniel.

More information about the libvir-list mailing list