Re: [PATCH] virsh: Free stream when shutdown console

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



于 2011年03月11日 23:19, Eric Blake 写道:
> On 03/11/2011 06:56 AM, Osier Yang wrote:
>> Otherwise connection of hypervisor driver will be leaked when
>> one shutdown the guest in console. e.g.
>>
>> [root@localhost]# init 0
>> ......
>> init: Re-executing /sbin/init
>> Halting system...
>> Power down.
>>
>> error: Failed to disconnect from the hypervisor, 1 leaked reference(s)
>> ---
>>   tools/console.c |    2 ++
>>   1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/tools/console.c b/tools/console.c
>> index b9dd268..01d8bd1 100644
>> --- a/tools/console.c
>> +++ b/tools/console.c
>> @@ -91,6 +91,8 @@ static void
>>   virConsoleShutdown(virConsolePtr con)
>>   {
>>       con->quit = true;
>> +    if (con->st)
>> +        virStreamFree(con->st);
>>       virStreamEventRemoveCallback(con->st);
> 
> Shouldn't you free the stream _after_ you remove the callback from it?
> Otherwise, you are referencing a freed stream.
> 
> ACK if you swap those two lines.
> 
Pushed with that fixed. Thanks.

Osier

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]