diff mbox

spice: call qemu_spice_set_passwd() during init

Message ID 1409918928-12324-1-git-send-email-marcandre.lureau@gmail.com
State New
Headers show

Commit Message

Marc-André Lureau Sept. 5, 2014, 12:08 p.m. UTC 
Don't call SPICE API directly to set password given in command line, but
use the internal API, saving password for later calls.

This solves losing password when changing expiration in qemu monitor.

https://bugzilla.redhat.com/show_bug.cgi?id=1138639
---
 ui/spice-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Gerd Hoffmann Sept. 5, 2014, 12:11 p.m. UTC | #1 
On Fr, 2014-09-05 at 14:08 +0200, Marc-André Lureau wrote:
> Don't call SPICE API directly to set password given in command line, but
> use the internal API, saving password for later calls.
> 
> This solves losing password when changing expiration in qemu monitor.

Makes sense.  Added to spice patch queue.

thanks,
  Gerd
diff mbox

Patch

diff --git a/ui/spice-core.c b/ui/spice-core.c
index 7bb91e6..f2e6521 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -733,7 +733,7 @@  void qemu_spice_init(void)
                              tls_ciphers);
     }
     if (password) {
-        spice_server_set_ticket(spice_server, password, 0, 0, 0);
+        qemu_spice_set_passwd(password, false, false);
     }
     if (qemu_opt_get_bool(opts, "sasl", 0)) {
         if (spice_server_set_sasl_appname(spice_server, "qemu") == -1 ||