[libvirt] [PATCH 2/2] vz: fix event handle leak in prlsdkHandlePerfEvent

[Maxim Nestratov]

08-Feb-17 19:07, Daniel P. Berrange пишет:

> On Wed, Feb 08, 2017 at 06:28:34PM +0300, Maxim Nestratov wrote:
>> When we happen to lose a domain but still get a performance event
>> for it, we should also free the event handle.
>>
>> Signed-off-by: Maxim Nestratov <mnestratov at virtuozzo.com>
>> ---
>>   src/vz/vz_sdk.c | 4 +++-
>>   1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/vz/vz_sdk.c b/src/vz/vz_sdk.c
>> index ec954dd..614dca1 100644
>> --- a/src/vz/vz_sdk.c
>> +++ b/src/vz/vz_sdk.c
>> @@ -2197,8 +2197,10 @@ prlsdkHandlePerfEvent(vzDriverPtr driver,
>>       virDomainObjPtr dom = NULL;
>>       vzDomObjPtr privdom = NULL;
>>   
>> -    if (!(dom = virDomainObjListFindByUUID(driver->domains, uuid)))
>> +    if (!(dom = virDomainObjListFindByUUID(driver->domains, uuid))) {
>> +        PrlHandle_Free(event);
>>           return;
>> +    }
>>   
>>       privdom = dom->privateData;
>>       PrlHandle_Free(privdom->stats);
> ACK
>
> Regards,
> Daniel

Pushed the series.

Thanks,
Maxim