[patch 1/2] spice-gtk: add PROP_USERNAME to spice session

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

 



Authentication with SASL needs username and password.

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

Index: new/gtk/spice-session.c
===================================================================
--- new.orig/gtk/spice-session.c	2013-10-23 12:25:12.000000000 +0200
+++ new/gtk/spice-session.c	2013-10-23 12:25:12.000000000 +0200
@@ -108,7 +108,8 @@
     PROP_NAME,
     PROP_CA,
     PROP_PROXY,
-    PROP_SECURE_CHANNELS
+    PROP_SECURE_CHANNELS,
+    PROP_USERNAME
 };
 
 /* signals */
@@ -229,6 +230,7 @@
     g_free(s->port);
     g_free(s->tls_port);
     g_free(s->password);
+    g_free(s->username);
     g_free(s->ca_file);
     g_free(s->ciphers);
     g_free(s->cert_subject);
@@ -272,7 +274,7 @@
 static int spice_uri_parse(SpiceSession *session, const char *original_uri)
 {
     SpiceSessionPrivate *s = SPICE_SESSION_GET_PRIVATE(session);
-    gchar *host = NULL, *port = NULL, *tls_port = NULL, *uri = NULL, *password = NULL;
+    gchar *host = NULL, *port = NULL, *tls_port = NULL, *uri = NULL, *password = NULL, *username = NULL;
     gchar *path = NULL;
     gchar *unescaped_path = NULL;
     gchar *authority = NULL;
@@ -358,6 +360,8 @@
             target_key = &port;
         } else if (g_str_equal(key, "tls-port")) {
             target_key = &tls_port;
+        } else if (g_str_equal(key, "username")) {
+            target_key = &username;
         } else if (g_str_equal(key, "password")) {
             target_key = &password;
             g_warning("password may be visible in process listings");
@@ -386,10 +390,12 @@
     g_free(s->port);
     g_free(s->tls_port);
     g_free(s->password);
+    g_free(s->username);
     s->host = host;
     s->port = port;
     s->tls_port = tls_port;
     s->password = password;
+    s->username = username;
     return 0;
 
 fail:
@@ -399,6 +405,7 @@
     g_free(port);
     g_free(tls_port);
     g_free(password);
+    g_free(username);
     return -1;
 }
 
@@ -422,6 +429,9 @@
     case PROP_TLS_PORT:
         g_value_set_string(value, s->tls_port);
 	break;
+    case PROP_USERNAME:
+        g_value_set_string(value, s->username);
+	break;
     case PROP_PASSWORD:
         g_value_set_string(value, s->password);
 	break;
@@ -529,7 +539,11 @@
         g_free(s->tls_port);
         s->tls_port = g_value_dup_string(value);
         break;
-    case PROP_PASSWORD:
+    case PROP_USERNAME:
+        g_free(s->username);
+        s->username = g_value_dup_string(value);
+        break;
+     case PROP_PASSWORD:
         g_free(s->password);
         s->password = g_value_dup_string(value);
         break;
@@ -688,6 +702,21 @@
                              G_PARAM_READWRITE |
                              G_PARAM_STATIC_STRINGS));
 
+   /**
+     * SpiceSession:username:
+     *
+     * username to use
+     *
+     **/
+    g_object_class_install_property
+        (gobject_class, PROP_USERNAME,
+         g_param_spec_string("username",
+                             "Username",
+                             "",
+                             NULL,
+                             G_PARAM_READWRITE |
+                             G_PARAM_STATIC_STRINGS));
+
     /**
      * SpiceSession:password:
      *
@@ -1199,6 +1228,7 @@
     g_warn_if_fail(c->host == NULL);
     g_warn_if_fail(c->tls_port == NULL);
     g_warn_if_fail(c->password == NULL);
+    g_warn_if_fail(c->username == NULL);
     g_warn_if_fail(c->ca_file == NULL);
     g_warn_if_fail(c->ciphers == NULL);
     g_warn_if_fail(c->cert_subject == NULL);
@@ -1210,6 +1240,7 @@
                  "host", &c->host,
                  "tls-port", &c->tls_port,
                  "password", &c->password,
+                 "username", &c->username,
                  "ca-file", &c->ca_file,
                  "ciphers", &c->ciphers,
                  "cert-subject", &c->cert_subject,
@@ -2037,6 +2068,15 @@
 }
 
 G_GNUC_INTERNAL
+const gchar* spice_session_get_username(SpiceSession *session)
+{
+    SpiceSessionPrivate *s = SPICE_SESSION_GET_PRIVATE(session);
+
+    g_return_val_if_fail(s != NULL, NULL);
+    return s->username;
+}
+
+G_GNUC_INTERNAL
 const gchar* spice_session_get_password(SpiceSession *session)
 {
     SpiceSessionPrivate *s = SPICE_SESSION_GET_PRIVATE(session);
Index: new/gtk/spice-session-priv.h
===================================================================
--- new.orig/gtk/spice-session-priv.h	2013-10-03 11:11:56.000000000 +0200
+++ new/gtk/spice-session-priv.h	2013-10-23 12:25:12.000000000 +0200
@@ -37,6 +37,7 @@
     char              *host;
     char              *port;
     char              *tls_port;
+    char              *username;
     char              *password;
     char              *ca_file;
     char              *ciphers;
@@ -135,6 +136,7 @@
 void spice_session_set_port(SpiceSession *session, int port, gboolean tls);
 void spice_session_get_pubkey(SpiceSession *session, guint8 **pubkey, guint *size);
 guint spice_session_get_verify(SpiceSession *session);
+const gchar* spice_session_get_username(SpiceSession *session);
 const gchar* spice_session_get_password(SpiceSession *session);
 const gchar* spice_session_get_host(SpiceSession *session);
 const gchar* spice_session_get_cert_subject(SpiceSession *session);
Index: new/gtk/spice-channel.c
===================================================================
--- new.orig/gtk/spice-channel.c	2013-10-23 12:25:12.000000000 +0200
+++ new/gtk/spice-channel.c	2013-10-24 07:26:17.000000000 +0200
@@ -1282,7 +1282,11 @@
         switch (interact[ninteract].id) {
         case SASL_CB_AUTHNAME:
         case SASL_CB_USER:
-            g_warn_if_reached();
+            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);
             break;
 
         case SASL_CB_PASS:
@@ -1351,6 +1355,8 @@
     const void *val;
     sasl_ssf_t ssf;
     sasl_callback_t saslcb[] = {
+        { .id = SASL_CB_USER },
+        { .id = SASL_CB_AUTHNAME },
         { .id = SASL_CB_PASS },
         { .id = 0 },
     };

_______________________________________________
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]