Re: [Qemu-devel] Re: [PATCH] vnc: Fix password expiration through 'change vnc ""'

[Anthony Liguori]

On 02/03/2011 11:02 AM, Daniel P. Berrange wrote:
> On Thu, Feb 03, 2011 at 10:35:51AM -0600, Anthony Liguori wrote:
>    
> > On 02/03/2011 10:29 AM, Daniel P. Berrange wrote:
> >      
> > > On Mon, Jan 31, 2011 at 02:43:19PM -0600, Anthony Liguori wrote:
> > >        
> > > > commit 52c18be9e99dabe295321153fda7fce9f76647ac introduced a regression in the
> > > > change vnc password command that changed the behavior of setting the VNC
> > > > password to an empty string from disabling login to disabling authentication.
> > > >
> > > > This commit refactors the code to eliminate this overloaded semantics in
> > > > vnc_display_password and instead introduces the vnc_display_disable_login.   The
> > > > monitor implementation then determines the behavior of an empty or missing
> > > > string.
> > > >          
> > > Personally I think this is a little overkill&   just reverting the
> > > original patch was fine, but from a functional POV your patch
> > > produces the same results, so I won't argue.
> > >        
> > For 0.15, I'd like to introduce a new set of commands such that we
> > don't multiplex the change command anymore.   This refactoring lays
> > the ground work for that.
> >
> > For instance, if you created a block device with the name 'vnc',
> > you'd get very unexpected results!  Multiplexing based on special
> > values on top of existing commands is pretty evil.
> >      
> Doesn't Gerd's 'set_password' command already replace the functionality
> of the 'change vnc' command. So we should likely declare 'change vnc'
> as deprecated in 0.14 and remove it in 0.16
>    

Yup.  But it doesn't let you disable login.  Since that was a feature of 
'change vnc', I think we need to provide a proper interface to do this.

Likewise, we need a new interface for changing the block device.  The 
way password setting is handled is fubar right now.

I've got some new commands documented in a git tree if you're interested.

http://repo.or.cz/w/qemu/aliguori.git/blob/refs/heads/glib:/qmp-schema.json

Regards,

Anthony Liguori

> Regards,
> Daniel
>