Re: [Xen-devel] [PATCH] libxl: don't require vncpasswd when empty

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Christoph Egger writes ("[Xen-devel] [PATCH] libxl: don't require vncpasswd 
when empty"):
> Attached patch fixes a regression introduced in c/s 22866:ed9ef3b07d2a:
> 
> Don't require a NULL-byte password when the guest config file
> has the entry
>      vncpasswd=''

I'm afraid I don't understand this patch.

If you say "vncpasswd=''" in the config file then info->vncpasswd
would be "" and therefore info->vncpasswd[0] would be '\0' so the
branch would not be taken and you would end up without ",password" on
the qemu command line.  The result should be that qemu should not
require a password.

Is that not what you would intend with such a config statement ?

Your patch seems to do the opposite of what you say it does.  It would
seem to make qemu require an empty string as a password, which is
daft.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel