[patch 1/1] fix SASL for mechanism using WANT_CLIENT_FIRST

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

 



Current code works with DIGEST-MD5, but not with PLAIN.

Signed-off-by: Dietmar Maurer <dietmar@xxxxxxxxxxx>

Index: new/gtk/spice-channel.c
===================================================================
--- new.orig/gtk/spice-channel.c	2013-10-22 09:04:23.000000000 +0200
+++ new/gtk/spice-channel.c	2013-10-22 09:40:10.000000000 +0200
@@ -1508,7 +1511,7 @@
 
     /* NB, distinction of NULL vs "" is *critical* in SASL */
     if (clientout) {
-        len += clientoutlen + 1;
+        len = clientoutlen + 1;
         spice_channel_write(channel, &len, sizeof(guint32));
         spice_channel_write(channel, clientout, len);
     } else {
@@ -1550,6 +1553,9 @@
      * Even if the server has completed, the client must *always* do at least one step
      * in this loop to verify the server isn't lying about something. Mutual auth */
     for (;;) {
+       if (complete && err == SASL_OK)
+            break;
+
     restep:
         err = sasl_client_step(saslconn,
                                serverin,

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]