[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] Fix changing password using monitor over VNC.
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH] Fix changing password using monitor over VNC. |
Date: |
Sat, 18 Apr 2009 10:41:04 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Zachary Amsden wrote:
> Jan Kiszka wrote:
>> Zachary Amsden wrote:
>>> Jan Kiszka wrote:
>>>> Zachary Amsden wrote:
>>>>> A simple segfault turned out to be a relatively complex fix.
>>>>>
>>>>> The monitor calls back to main_loop_wait() to wait for the completion
>>>>> of the password change event; this results in a nested invocation of
>>>> This is no longer true with trunk as this nasty blocking password
>>>> reading has been converted into an async operation. Is your patch
>>>> required nevertheless? Or is this band-aid for stable?
>>> I confirmed the bug, verified the fix on a git tree sync'd today from
>>> git://git.savannah.nongnu.org/qemu.git
>>>
>>> let me know if this is not the trunk.
>> Nope, it's outdated. Last check in is from end of January this year.
>>
>> There are several up-to-date git mirrors, one e.g. at kernel.org:
>> git://git.kernel.org/pub/scm/virt/qemu/qemu.git. However, I prefer to do
>> my own mirroring with git svn.
>>
>>> Sorry, as I am new to qemu development.
>> No problem. The issue you found is probably still relevant for stable
>> 0.10.x, so please consider this branch as well.
>
> The patch applies as is to stable branch.
>
> And this new patch fixes two bugs on trunk:
>
> 1) Changing VNC password should set password auth if it wasn't yet enabled
> 2) Trying to change VNC password on a SDL-only display gives a segfault.
>
I thought I fixed something like 2) along the monitor rework, but
obviously not every case (VNC is not in daily use here).
However, your patch requires standard format:
[PATCH] <subject> (if it starts with 'Re:', it risks to be overseen)
<description>
Signed-off-by: ...
<diff>
Thanks,
Jan
signature.asc
Description: OpenPGP digital signature