populate 'send keys' menu and wire up key sending mechanism
[ovirt-viewer.git] / main.c
diff --git a/main.c b/main.c
index cf37b31..356d379 100644 (file)
--- a/main.c
+++ b/main.c
 
 #include <glib.h>
 #include <gtk/gtk.h>
+#include <gdk/gdkkeysyms.h>
 #include <vncdisplay.h>
 
+#ifdef HAVE_NETDB_H
+#include <netdb.h>
+#endif
+
+#ifdef HAVE_NETINET_IN_H
+#include <netinet/in.h>
+#endif
+
 #ifdef HAVE_SYS_SOCKET_H
 #include <sys/socket.h>
 #endif
@@ -50,16 +59,38 @@ gboolean debug = 0;
  * configure or on the command line.
  */
 const char *cainfo = CAINFO;
-gboolean check_cert = TRUE;
+gboolean check_cert = FALSE; // do we want this enabled by default ?
+                      // would require a CA by default (self-signed wont work)
+                      // (don't set to true, change var/flag to no_check_cert)
+
+/* The WUI thread has updated the vm list.  Here in the main thread
+* we keep our own copy of the vmlist.
+*/
+static GSList *vmlist = NULL;
 
 /* Private functions. */
 static void start_ui (void);
 static GtkWidget *menu_item_new (int which_menu);
+static void refresh_menu_vm_list (GtkWidget *, gpointer);
+static void connect_to_wui_on_enter (GtkWidget *, gpointer);
 static void connect_to_wui (GtkWidget *, gpointer);
+static void send_key_to_vm (GtkWidget *widget, gpointer _keyComboDef);
+static void login_to_wui_on_enter (GtkWidget *, gpointer);
 static void login_to_wui (GtkWidget *, gpointer);
 static gboolean delete_event (GtkWidget *widget, GdkEvent *event, gpointer data);
 static void destroy (GtkWidget *widget, gpointer data);
 static void clear_connectmenu (void);
+static void help_about (GtkWidget *menu);
+static void viewer_shutdown (GtkWidget *src, void *dummy, GtkWidget *vnc);
+#if 0
+static void viewer_quit (GtkWidget *src, GtkWidget *vnc);
+#endif
+static void viewer_connected (GtkWidget *vnc);
+static void viewer_initialized (GtkWidget *vnc, GtkWidget *data);
+static void viewer_disconnected (GtkWidget *vnc);
+static void viewer_credential (GtkWidget *vnc, GValueArray *credList);
+static int viewer_open_vnc_socket (const char *vnchost, int vncport);
+static void add_vm_to_connectmenu (gpointer _vm, gpointer data);
 
 /* For any widgets accessed from multiple functions. */
 static GtkWidget *window;
@@ -76,6 +107,7 @@ static GtkWidget *login_area;
 static GtkWidget *la_username;
 static GtkWidget *la_password;
 static GtkWidget *la_button;
+static GtkWidget *la_error;
 static GtkWidget *notebook;
 static GtkWidget *statusbar;
 static guint statusbar_ctx;
@@ -105,9 +137,42 @@ static struct menuItem menuItems[] = {
   { HELP_MENU, NULL, "_Help", "Help" }
 };
 
+#define MAX_KEY_COMBO 3
+ struct keyComboDef {
+ guint keys[MAX_KEY_COMBO];
+ guint nkeys;
+ const char *label;
+ };
+
+#define NUM_KEY_COMBOS 17
+static struct keyComboDef keyCombos[] = {
+ { { GDK_Control_L, GDK_Alt_L, GDK_Delete }, 3, "Ctrl+Alt+Del"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_BackSpace }, 3, "Ctrl+Alt+Backspace"},
+ { {}, 0, "" },
+ { { GDK_Control_L, GDK_Alt_L, GDK_F1 }, 3, "Ctrl+Alt+F1"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F2 }, 3, "Ctrl+Alt+F2"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F3 }, 3, "Ctrl+Alt+F3"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F4 }, 3, "Ctrl+Alt+F4"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F5 }, 3, "Ctrl+Alt+F5"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F6 }, 3, "Ctrl+Alt+F6"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F7 }, 3, "Ctrl+Alt+F7"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F8 }, 3, "Ctrl+Alt+F8"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F5 }, 3, "Ctrl+Alt+F9"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F6 }, 3, "Ctrl+Alt+F10"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F7 }, 3, "Ctrl+Alt+F11"},
+ { { GDK_Control_L, GDK_Alt_L, GDK_F8 }, 3, "Ctrl+Alt+F12"},
+ { {}, 0, "" },
+ { { GDK_Print }, 1, "PrintScreen"},
+};
+
 /* Window title. */
 static const char *title = "oVirt Viewer";
 
+// when running vm
+// 47 chars
+static const char *title_vm =
+   "oVirt Viewer: (ctrl+alt to grab/release mouse) ";
+
 /* Gtk widget styles.  Avoid installation hassles by keeping this
  * inside the binary.  It can still be overridden by the user (who
  * will do that?)
@@ -130,7 +195,7 @@ static const GOptionEntry options[] = {
   { "cainfo", 0, 0, G_OPTION_ARG_STRING, &cainfo,
     "set the path of the CA certificate bundle", NULL },
   { "check-certificate", 0, 0, G_OPTION_ARG_NONE, &check_cert,
-    "if --no-check-certificate is passed we don't check the SSL certificate of the server", NULL },
+    "check the SSL certificate of the server", NULL },
   { "debug", 'd', 0, G_OPTION_ARG_NONE, &debug,
     "turn on debugging messages", NULL },
   { "version", 'V', 0, G_OPTION_ARG_NONE, &print_version,
@@ -210,20 +275,25 @@ main (int argc, char *argv[])
 static void
 start_ui (void)
 {
+  int i;
   GtkWidget *vbox;
   GtkWidget *menubar;
   GtkWidget *view;
   GtkWidget *viewmenu;
   GtkWidget *sendkey;
   GtkWidget *sendkeymenu;
+  GtkWidget *sendkeymenuitem;
   GtkWidget *wind;
   GtkWidget *windmenu;
   GtkWidget *help;
   GtkWidget *helpmenu;
+  GtkWidget *about;
   GtkWidget *ca_vbox;
   GtkWidget *ca_hbox;
   GtkWidget *ca_label;
+  GtkWidget *la_vbox;
   GtkWidget *la_hbox;
+  GtkWidget *la_label;
 
   DEBUG ("creating viewer windows and menus");
 
@@ -268,6 +338,9 @@ start_ui (void)
   refresh_vmlist_separator = gtk_separator_menu_item_new ();
   g_object_ref (refresh_vmlist_separator);
 
+  g_signal_connect (G_OBJECT (refresh_vmlist), "activate",
+                   G_CALLBACK (refresh_menu_vm_list), NULL);
+
 #if 0
   screenshot = gtk_menu_item_new_with_mnemonic ("_Screenshot");
   gtk_menu_append (GTK_MENU (filemenu), screenshot);
@@ -283,6 +356,13 @@ start_ui (void)
   sendkeymenu = gtk_menu_new ();
   gtk_menu_item_set_submenu (GTK_MENU_ITEM (sendkey), sendkeymenu);
 
+  for(i = 0; i < NUM_KEY_COMBOS; ++i){
+    sendkeymenuitem = gtk_menu_item_new_with_label (keyCombos[i].label);
+    gtk_menu_append (GTK_MENU (sendkeymenu), sendkeymenuitem);
+    g_signal_connect (G_OBJECT (sendkeymenuitem), "activate",
+                      G_CALLBACK (send_key_to_vm), &(keyCombos[i]));
+  }
+
   wind = menu_item_new (WINDOW_MENU);
   windmenu = gtk_menu_new ();
   gtk_menu_item_set_submenu (GTK_MENU_ITEM (wind), windmenu);
@@ -291,6 +371,10 @@ start_ui (void)
   helpmenu = gtk_menu_new ();
   gtk_menu_item_set_submenu (GTK_MENU_ITEM (help), helpmenu);
 
+  about = gtk_image_menu_item_new_from_stock(GTK_STOCK_ABOUT, NULL);
+  gtk_menu_append(GTK_MENU(helpmenu), about);
+  g_signal_connect(about, "activate", GTK_SIGNAL_FUNC (help_about), NULL);
+
   gtk_menu_bar_append (GTK_MENU_BAR (menubar), connectitem);
   gtk_menu_bar_append (GTK_MENU_BAR (menubar), view);
   gtk_menu_bar_append (GTK_MENU_BAR (menubar), sendkey);
@@ -315,24 +399,36 @@ start_ui (void)
 
   gtk_widget_set_name (connection_area, "ovirt-viewer-connection-area");
 
+  g_signal_connect (G_OBJECT (ca_hostname), "key-release-event",
+                   G_CALLBACK (connect_to_wui_on_enter), NULL);
   g_signal_connect (G_OBJECT (ca_button), "clicked",
                    G_CALLBACK (connect_to_wui), NULL);
 
   login_area = gtk_event_box_new ();
+  la_vbox = gtk_vbox_new (FALSE, 0);
   la_hbox = gtk_hbox_new (FALSE, 0);
+  la_label = gtk_label_new ("Username / password:");
   la_username = gtk_entry_new ();
   gtk_entry_set_width_chars (GTK_ENTRY (la_username), 12);
   la_password = gtk_entry_new ();
   gtk_entry_set_width_chars (GTK_ENTRY (la_password), 12);
   gtk_entry_set_visibility (GTK_ENTRY (la_password), FALSE);
   la_button = gtk_button_new_with_label ("Login");
+  la_error = gtk_label_new (NULL);
   gtk_container_add (GTK_CONTAINER (la_hbox), la_username);
   gtk_container_add (GTK_CONTAINER (la_hbox), la_password);
   gtk_container_add (GTK_CONTAINER (la_hbox), la_button);
-  gtk_container_add (GTK_CONTAINER (login_area), la_hbox);
+  gtk_box_pack_start (GTK_BOX (la_vbox), la_label,     TRUE,  FALSE, 4);
+  gtk_box_pack_start (GTK_BOX (la_vbox), la_hbox,     TRUE,  FALSE, 4);
+  gtk_box_pack_start (GTK_BOX (la_vbox), la_error,    TRUE,  FALSE, 4);
+  gtk_container_add (GTK_CONTAINER (login_area), la_vbox);
 
   gtk_widget_set_name (login_area, "ovirt-viewer-login-area");
 
+  g_signal_connect (G_OBJECT (la_username), "key-release-event",
+                   G_CALLBACK (login_to_wui_on_enter), NULL);
+  g_signal_connect (G_OBJECT (la_password), "key-release-event",
+                   G_CALLBACK (login_to_wui_on_enter), NULL);
   g_signal_connect (G_OBJECT (la_button), "clicked",
                    G_CALLBACK (login_to_wui), NULL);
 
@@ -408,6 +504,57 @@ destroy (GtkWidget *widget, gpointer data)
 }
 
 static void
+help_about (GtkWidget *menu)
+{
+  GtkWidget *about;
+  const char *authors[] = {
+    "Richard W.M. Jones <rjones@redhat.com>",
+    "Daniel P. Berrange <berrange@redhat.com>",
+    NULL
+  };
+
+  about = gtk_about_dialog_new();
+
+  gtk_about_dialog_set_name(GTK_ABOUT_DIALOG(about), "oVirt Viewer");
+  gtk_about_dialog_set_version(GTK_ABOUT_DIALOG(about), VERSION);
+  gtk_about_dialog_set_website(GTK_ABOUT_DIALOG(about), "http://ovirt.org/");
+  gtk_about_dialog_set_website_label(GTK_ABOUT_DIALOG(about), "oVirt website");
+  gtk_about_dialog_set_authors(GTK_ABOUT_DIALOG(about), authors);
+  gtk_about_dialog_set_license(GTK_ABOUT_DIALOG(about), 
+ "This program is free software; you can redistribute it and/or modify\n" \
+ "it under the terms of the GNU General Public License as published by\n" \
+ "the Free Software Foundation; either version 2 of the License, or\n" \
+ "(at your option) any later version.\n" \
+ "\n" \
+ "This program is distributed in the hope that it will be useful,\n" \
+ "but WITHOUT ANY WARRANTY; without even the implied warranty of\n" \
+ "MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\n" \
+ "GNU General Public License for more details.\n" \
+ "\n" \
+ "You should have received a copy of the GNU General Public License\n" \
+ "along with this program; if not, write to the Free Software\n" \
+ "Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA\n");
+
+  gtk_dialog_run(GTK_DIALOG(about));
+  gtk_widget_destroy(about);
+}
+
+static void
+refresh_menu_vm_list(GtkWidget *widget, gpointer data)
+{
+   wui_thread_send_refresh_vm_list();
+}
+
+static void
+connect_to_wui_on_enter (GtkWidget *widget, gpointer data)
+{
+  // if key released was not 'enter' key
+  if(((GdkEventKey *)data)->type == GDK_KEY_RELEASE && (((GdkEventKey *)data)->keyval & 0xFF) != 13 ) return;
+
+  connect_to_wui(widget, data);
+}
+
+static void
 connect_to_wui (GtkWidget *widget, gpointer data)
 {
   const char *hostname;
@@ -426,6 +573,15 @@ connect_to_wui (GtkWidget *widget, gpointer data)
 }
 
 static void
+login_to_wui_on_enter (GtkWidget *widget, gpointer data)
+{
+  // if key released was not 'enter' key
+  if(((GdkEventKey *)data)->type == GDK_KEY_RELEASE && (((GdkEventKey *)data)->keyval & 0xFF) != 13 ) return;
+
+  login_to_wui(widget, data);
+}
+
+static void
 login_to_wui (GtkWidget *widget, gpointer data)
 {
   const char *username, *password;
@@ -447,10 +603,12 @@ connect_to_vm (GtkWidget *widget, gpointer _vm)
 {
   struct vm *vm = (struct vm *) _vm;
   int n = gtk_notebook_get_n_pages (GTK_NOTEBOOK (notebook));
-  int i, uuidlen, len;
+  int i, uuidlen, len, fd;
   GtkWidget *child;
   const char *label;
+  const char* hostname;
   char *label2;
+  char new_title[97]; // 47 chars for title + 50 for vm name
 
   DEBUG ("searching tabs for uuid %s", vm->uuid);
 
@@ -471,16 +629,65 @@ connect_to_vm (GtkWidget *widget, gpointer _vm)
 
   DEBUG ("not found, creating new tab");
 
+  // before we open vnc connection, make sure vm is running
+  gboolean vm_running = main_vmlist_has_running_vm(vm);
+
+  if(!vm_running){
+    main_status_error (g_strdup ("VM not running"));
+    return;
+  }
+
   /* This VM isn't in the notebook already, so create a new console. */
-  len = strlen (vm->description) + 1 + strlen (vm->uuid) + 1;
-  label2 = g_alloca (len);
-  snprintf (label2, len, "%s %s", vm->description, vm->uuid);
+  hostname = gtk_entry_get_text (GTK_ENTRY (ca_hostname));
+  fd = viewer_open_vnc_socket(hostname, vm->forward_vnc_port);
+  if (fd == -1) return;                /* We've already given an error. */
+
+  child = vnc_display_new ();
+  if (! vnc_display_open_fd (VNC_DISPLAY (child), fd)) {
+    main_status_error (g_strdup ("internal error in Gtk-VNC widget"));
+    return;
+  }
 
-  child = gtk_label_new (label2); /* XXX */
+  main_status_error(g_strdup(""));
+
+  for(i = 0; i < 47; ++i) new_title[i] = title_vm[i];
+  for(i = 0; i < 50; ++i){
+     if(vm->description[i] == '\0') break;
+     new_title[47 + i] = vm->description[i];
+  }
+  new_title[i < 50 ? i+47 : 96] = '\0';
+  gtk_window_set_title (GTK_WINDOW (window), new_title);
+
+  /*
+  gtk_signal_connect(GTK_OBJECT(child), "vnc-pointer-grab",
+                    GTK_SIGNAL_FUNC(viewer_grab), window);
+  gtk_signal_connect(GTK_OBJECT(child), "vnc-pointer-ungrab",
+                    GTK_SIGNAL_FUNC(viewer_ungrab), window);
+  */
+
+  gtk_signal_connect(GTK_OBJECT(child), "delete-event",
+                    GTK_SIGNAL_FUNC(viewer_shutdown), child);
+
+  gtk_signal_connect(GTK_OBJECT(child), "vnc-connected",
+                    GTK_SIGNAL_FUNC(viewer_connected), NULL);
+  gtk_signal_connect(GTK_OBJECT(child), "vnc-initialized",
+                    GTK_SIGNAL_FUNC(viewer_initialized), NULL);
+  gtk_signal_connect(GTK_OBJECT(child), "vnc-disconnected",
+                    GTK_SIGNAL_FUNC(viewer_disconnected), NULL);
+
+  g_signal_connect(GTK_OBJECT(child), "vnc-auth-credential",
+                  GTK_SIGNAL_FUNC(viewer_credential), NULL);
 
   /* NB. We have to do this before adding it to the notebook. */
   gtk_widget_show (child);
 
+  /* Choose a tab label, which MUST end with the uuid string, since
+   * we use the tab label to store uuid.
+   */
+  len = strlen (vm->description) + 1 + strlen (vm->uuid) + 1;
+  label2 = g_alloca (len);
+  snprintf (label2, len, "%s %s", vm->description, vm->uuid);
+
   i = gtk_notebook_append_page (GTK_NOTEBOOK (notebook), child, NULL);
   gtk_notebook_set_tab_label_text (GTK_NOTEBOOK (notebook), child, label2);
   gtk_notebook_set_current_page (GTK_NOTEBOOK (notebook), i);
@@ -488,6 +695,230 @@ connect_to_vm (GtkWidget *widget, gpointer _vm)
   DEBUG ("finished creating new tab");
 }
 
+/* Send key to a virtual machine.  This callback is called from the
+ * send key menu.  If finds vm in focus, and sends specified key to it
+ */
+static void
+send_key_to_vm (GtkWidget *widget, gpointer _keyComboDef)
+{
+  GtkWidget* viewer;
+  struct keyComboDef* key_combo = (struct keyComboDef*) _keyComboDef;
+  int c = gtk_notebook_get_current_page (GTK_NOTEBOOK (notebook));
+
+  if(c < 0){
+    return;
+  }
+
+  viewer = gtk_notebook_get_nth_page(GTK_NOTEBOOK (notebook), c);
+  if(viewer != NULL){
+    DEBUG ("sending keys to vm");
+    vnc_display_send_keys(VNC_DISPLAY(viewer),
+                          key_combo->keys,
+                          key_combo->nkeys);
+  }
+}
+
+
+/*
+static void viewer_grab(GtkWidget *vnc, GtkWidget *window)
+{
+       int i;
+
+       viewer_set_title(VNC_DISPLAY(vnc), window, TRUE);
+
+       for (i = 0 ; i < LAST_MENU; i++) {
+               gtk_label_set_text_with_mnemonic(GTK_LABEL(menuItems[i].label), menuItems[i].grabbed_text);
+       }
+}
+
+static void viewer_ungrab(GtkWidget *vnc, GtkWidget *window)
+{
+       int i;
+
+       viewer_set_title(VNC_DISPLAY(vnc), window, FALSE);
+
+       for (i = 0 ; i < LAST_MENU; i++) {
+               gtk_label_set_text_with_mnemonic(GTK_LABEL(menuItems[i].label), menuItems[i].ungrabbed_text);
+       }
+}
+*/
+
+static void
+viewer_shutdown (GtkWidget *src, void *dummy, GtkWidget *vnc)
+{
+  vnc_display_close (VNC_DISPLAY(vnc));
+
+  /* Just close the notebook tab for now. XXX */
+  gtk_notebook_remove_page (GTK_NOTEBOOK (notebook),
+                           gtk_notebook_page_num (GTK_NOTEBOOK (notebook),
+                                                  vnc));
+}
+
+#if 0
+static void
+viewer_quit (GtkWidget *src, GtkWidget *vnc)
+{
+  viewer_shutdown (src, NULL, vnc);
+}
+#endif
+
+static void
+viewer_connected (GtkWidget *vnc)
+{
+  DEBUG ("Connected to server");
+}
+
+static void
+viewer_initialized (GtkWidget *vnc, GtkWidget *data)
+{
+  DEBUG ("Connection initialized");
+}
+
+static void
+viewer_disconnected (GtkWidget *vnc)
+{
+  DEBUG ("Disconnected from server");
+}
+
+static void
+viewer_credential (GtkWidget *vnc, GValueArray *credList)
+{
+        GtkWidget *dialog = NULL;
+        int response;
+        unsigned int i, prompt = 0;
+        const char **data;
+
+        DEBUG ("Got credential request for %d credential(s)",
+              credList->n_values);
+
+        data = g_new0(const char *, credList->n_values);
+
+        for (i = 0 ; i < credList->n_values ; i++) {
+                GValue *cred = g_value_array_get_nth(credList, i);
+                switch (g_value_get_enum(cred)) {
+                case VNC_DISPLAY_CREDENTIAL_USERNAME:
+                case VNC_DISPLAY_CREDENTIAL_PASSWORD:
+                        prompt++;
+                        break;
+                case VNC_DISPLAY_CREDENTIAL_CLIENTNAME:
+                        data[i] = "libvirt";
+                default:
+                        break;
+                }
+        }
+
+        if (prompt) {
+                GtkWidget **label, **entry, *box, *vbox;
+                int row;
+                dialog = gtk_dialog_new_with_buttons("Authentication required",
+                                                     NULL,
+                                                     0,
+                                                     GTK_STOCK_CANCEL,
+                                                     GTK_RESPONSE_CANCEL,
+                                                     GTK_STOCK_OK,
+                                                     GTK_RESPONSE_OK,
+                                                     NULL);
+                gtk_dialog_set_default_response(GTK_DIALOG(dialog), GTK_RESPONSE_OK);
+
+                box = gtk_table_new(credList->n_values, 2, FALSE);
+                label = g_new(GtkWidget *, prompt);
+                entry = g_new(GtkWidget *, prompt);
+
+                for (i = 0, row =0 ; i < credList->n_values ; i++) {
+                        GValue *cred = g_value_array_get_nth(credList, i);
+                        switch (g_value_get_enum(cred)) {
+                        case VNC_DISPLAY_CREDENTIAL_USERNAME:
+                                label[row] = gtk_label_new("Username:");
+                                break;
+                        case VNC_DISPLAY_CREDENTIAL_PASSWORD:
+                                label[row] = gtk_label_new("Password:");
+                                break;
+                        default:
+                                continue;
+                        }
+                        entry[row] = gtk_entry_new();
+                        if (g_value_get_enum(cred) == VNC_DISPLAY_CREDENTIAL_PASSWORD)
+                                gtk_entry_set_visibility(GTK_ENTRY(entry[row]), FALSE);
+
+                        gtk_table_attach(GTK_TABLE(box), label[i], 0, 1, row, row+1, GTK_SHRINK, GTK_SHRINK, 3, 3);
+                        gtk_table_attach(GTK_TABLE(box), entry[i], 1, 2, row, row+1, GTK_SHRINK, GTK_SHRINK, 3, 3);
+                        row++;
+                }
+
+                vbox = gtk_bin_get_child(GTK_BIN(dialog));
+                gtk_container_add(GTK_CONTAINER(vbox), box);
+
+                gtk_widget_show_all(dialog);
+                response = gtk_dialog_run(GTK_DIALOG(dialog));
+                gtk_widget_hide(GTK_WIDGET(dialog));
+
+                if (response == GTK_RESPONSE_OK) {
+                        for (i = 0, row = 0 ; i < credList->n_values ; i++) {
+                                GValue *cred = g_value_array_get_nth(credList, i);
+                                switch (g_value_get_enum(cred)) {
+                                case VNC_DISPLAY_CREDENTIAL_USERNAME:
+                                case VNC_DISPLAY_CREDENTIAL_PASSWORD:
+                                        data[i] = gtk_entry_get_text(GTK_ENTRY(entry[row]));
+                                        break;
+                                }
+                        }
+                }
+        }
+
+        for (i = 0 ; i < credList->n_values ; i++) {
+                GValue *cred = g_value_array_get_nth(credList, i);
+                if (data[i]) {
+                        if (vnc_display_set_credential(VNC_DISPLAY(vnc),
+                                                       g_value_get_enum(cred),
+                                                       data[i])) {
+                                DEBUG("Failed to set credential type %d",
+                                     g_value_get_enum(cred));
+                                vnc_display_close(VNC_DISPLAY(vnc));
+                        }
+                } else {
+                        DEBUG("Unsupported credential type %d",
+                             g_value_get_enum(cred));
+                        vnc_display_close(VNC_DISPLAY(vnc));
+                }
+        }
+
+        g_free(data);
+        if (dialog)
+                gtk_widget_destroy(GTK_WIDGET(dialog));
+}
+
+#if defined(HAVE_SOCKET) && defined(HAVE_CONNECT) && defined(HAVE_HTONS) && defined(HAVE_GETHOSTBYNAME)
+
+static int 
+viewer_open_vnc_socket(const char* vnchost, int vncport)
+{
+  int socketfd;
+  struct hostent *serv;
+  struct sockaddr_in serv_addr;
+
+  socketfd = socket(PF_INET, SOCK_STREAM, 0);
+  if(socketfd < 0){
+      return -1;
+  }
+
+  serv = gethostbyname(vnchost);
+  if(serv == NULL){
+      return -1;
+  }
+
+  serv_addr.sin_family = PF_INET;
+  serv_addr.sin_port = htons(vncport);
+  serv_addr.sin_addr.s_addr = ((struct in_addr *)(serv->h_addr))->s_addr; 
+
+  if (connect(socketfd,(struct sockaddr *)&serv_addr,sizeof(serv_addr)) < 0){
+      return -1;
+  }
+
+  return socketfd;
+}
+
+#endif /* defined(HAVE_SOCKET) && defined(HAVE_CONNECT) && defined(HAVE_HTONS) && defined(HAVE_GETHOSTBYNAME) */
+
 /* Remove all menu items from the Connect menu. */
 static void
 remove_menu_item (GtkWidget *menu_item, gpointer data)
@@ -621,9 +1052,7 @@ main_login_error (gpointer _str)
   DEBUG ("login error: %s", str);
   ASSERT_IS_MAIN_THREAD ();
 
-  /*
   gtk_label_set_text (GTK_LABEL (la_error), str);
-  */
   g_free (str);
 
   return FALSE;
@@ -647,13 +1076,6 @@ main_status_error (gpointer _str)
   return FALSE;
 }
 
-/* The WUI thread has updated the vm list.  Here in the main thread
- * we keep our own copy of the vmlist.
- */
-static GSList *vmlist = NULL;
-
-static void add_vm_to_connectmenu (gpointer _vm, gpointer data);
-
 gboolean
 main_vmlist_updated (gpointer data)
 {
@@ -694,6 +1116,9 @@ add_vm_to_connectmenu (gpointer _vm, gpointer data)
   struct vm *vm = (struct vm *) _vm;
   GtkWidget *item;
 
+  // TODO only present running vms ?
+  // if(vm->state == "running")
+
   DEBUG ("adding %s to Connect menu", vm->description);
 
   item = gtk_menu_item_new_with_label (vm->description);