[Spice-commits] src/spice-channel.c

Pavel Grunt pgrunt at kemper.freedesktop.org
Wed Jun 15 13:25:48 UTC 2016


 src/spice-channel.c |    9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

New commits:
commit fb8e51667b9fa63497e413fb4fd484d23b772788
Author: Alexander Bokovoy <abokovoy at redhat.com>
Date:   Mon Jun 6 18:04:59 2016 +0200

    sasl: fix SASL GSSAPI by allowing NULL username
    
    SASL GSSAPI module will try to negotiate authentication based on the
    credentials in the default credentials cache. It does not matter if
    SPICE knows username or not as SASL negotiation will pass through the
    discovered name from the GSSAPI module.
    
    Signed-off-by: Alexander Bokovoy <abokovoy at redhat.com>
    Acked-by: Fabiano FidĂȘncio <fidencio at redhat.com>

diff --git a/src/spice-channel.c b/src/spice-channel.c
index c6e548d..0eb0e61 100644
--- a/src/spice-channel.c
+++ b/src/spice-channel.c
@@ -1387,11 +1387,10 @@ spice_channel_gather_sasl_credentials(SpiceChannel *channel,
         switch (interact[ninteract].id) {
         case SASL_CB_AUTHNAME:
         case SASL_CB_USER:
-            if (spice_session_get_username(c->session) == NULL)
-                return FALSE;
-
-            interact[ninteract].result =  spice_session_get_username(c->session);
-            interact[ninteract].len = strlen(interact[ninteract].result);
+            if (spice_session_get_username(c->session) != NULL) {
+                interact[ninteract].result =  spice_session_get_username(c->session);
+                interact[ninteract].len = strlen(interact[ninteract].result);
+            }
             break;
 
         case SASL_CB_PASS:


More information about the Spice-commits mailing list