1 files changed, 45 insertions, 27 deletions

diff --git a/src/peerinfo/gnunet-peerinfo-gtk.c b/src/peerinfo/gnunet-peerinfo-gtk.c
index 13b820eb..6d8569e8 100644
--- a/src/peerinfo/gnunet-peerinfo-gtk.c
+++ b/src/peerinfo/gnunet-peerinfo-gtk.c
@@ -1,6 +1,6 @@
 /*
      This file is part of GNUnet.
-     (C) 2010 Christian Grothoff (and other contributing authors)
+     (C) 2010, 2012 Christian Grothoff (and other contributing authors)
 
      GNUnet is free software; you can redistribute it and/or modify
      it under the terms of the GNU General Public License as published
@@ -92,6 +92,10 @@ struct PeerInfo
    */
   struct GNUNET_TRANSPORT_AddressToStringContext *tos;
 
+  /**
+   * Did we get any address?
+   */
+  int got_address;
 };
 
 
@@ -216,7 +220,6 @@ peer_address_string_cb (void *cls, const char *address)
   const char *colon;
   const char *dot;
 
-  info->tos = NULL;
   ls = GTK_LIST_STORE (get_object ("GNUNET_PEERINFO_GTK_list_store"));
   tm = GTK_TREE_MODEL (ls);
   path = gtk_tree_row_reference_get_path (info->rr);
@@ -226,34 +229,35 @@ peer_address_string_cb (void *cls, const char *address)
   if (NULL == address)
   {
     /* error */
-    gtk_list_store_set (ls, &iter, 
-                        PEERINFO_MC_NUMBER_OF_ADDRESSES, (guint) 1, 
-                        PEERINFO_MC_COUNTRY_NAME, NULL, 
-                        PEERINFO_MC_COUNTRY_FLAG, NULL, 
-                        PEERINFO_MC_ADDRESS_AS_STRING, "<error>", 
-                        -1);
+    if (GNUNET_NO == info->got_address)
+      gtk_list_store_set (ls, &iter, 
+                          PEERINFO_MC_NUMBER_OF_ADDRESSES, (guint) 1, 
+                          PEERINFO_MC_COUNTRY_NAME, NULL, 
+                          PEERINFO_MC_COUNTRY_FLAG, NULL, 
+                          PEERINFO_MC_ADDRESS_AS_STRING, "<no address>", 
+                          -1);
+    info->tos = NULL;
+    return;
   }
-  else
+  /* last address, store information in model */
+  country = NULL;
+  colon = strstr (address, ":");
+  if (NULL != colon)
   {
-    /* last address, store information in model */
-    country = NULL;
-    colon = strstr (address, ":");
-    if (NULL != colon)
-    {
-      for (dot = colon - 1; dot != address; dot--)
-        if ('.' == *dot)
-          break;
+    for (dot = colon - 1; dot != address; dot--)
       if ('.' == *dot)
-        country = GNUNET_strndup (&dot[1], (colon - dot) - 1);
-    }
-    gtk_list_store_set (ls, &iter, 
-                        PEERINFO_MC_NUMBER_OF_ADDRESSES, 1,
-                        PEERINFO_MC_COUNTRY_NAME, country, 
-                        PEERINFO_MC_COUNTRY_FLAG, GNUNET_PEERINFO_GTK_get_flag (country), 
-                        PEERINFO_MC_ADDRESS_AS_STRING, address, 
-                        -1);
-    GNUNET_free (country);
+        break;
+    if ('.' == *dot)
+      country = GNUNET_strndup (&dot[1], (colon - dot) - 1);
   }
+  gtk_list_store_set (ls, &iter, 
+                      PEERINFO_MC_NUMBER_OF_ADDRESSES, 1,
+                      PEERINFO_MC_COUNTRY_NAME, country, 
+                      PEERINFO_MC_COUNTRY_FLAG, GNUNET_PEERINFO_GTK_get_flag (country), 
+                      PEERINFO_MC_ADDRESS_AS_STRING, address, 
+                      -1);
+  GNUNET_free (country);
+  info->got_address = GNUNET_YES;
 }
 
 
@@ -293,6 +297,7 @@ peer_address_cb (void *cls, const struct GNUNET_PeerIdentity *peer,
   }
   if (NULL != info->tos)
     GNUNET_TRANSPORT_address_to_string_cancel (info->tos);
+  info->got_address = GNUNET_NO;
   info->tos =
       GNUNET_TRANSPORT_address_to_string (get_configuration (), address,
                                           GNUNET_NO,
@@ -343,7 +348,7 @@ peerinfo_processor (void *cls, const struct GNUNET_PeerIdentity *peer,
                         PEERINFO_MC_COUNTRY_FLAG, NULL,
                         PEERINFO_MC_BANDWIDTH_IN, (guint64) 0,
                         PEERINFO_MC_BANDWIDTH_OUT, (guint64) 0,
-                        PEERINFO_MC_ADDRESS_AS_STRING, "",
+                        PEERINFO_MC_ADDRESS_AS_STRING, "<disconnected>",
                         -1);
     path = gtk_tree_model_get_path (tm, &iter);
     info = GNUNET_malloc (sizeof (struct PeerInfo));
@@ -406,6 +411,9 @@ status_cb (void *cls,
 
 /**
  * Callback invoked if the application is supposed to exit.
+ *
+ * @param object 
+ * @param user_data unused
  */
 void
 GNUNET_PEERINFO_GTK_quit_cb (GObject * object, gpointer user_data)
@@ -419,6 +427,9 @@ GNUNET_PEERINFO_GTK_quit_cb (GObject * object, gpointer user_data)
 /**
  * Actual main function run right after GNUnet's scheduler
  * is initialized.  Initializes up GTK and Glade.
+ *
+ * @param cls NULL
+ * @param tc schedule context
  */
 static void
 run (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
@@ -460,6 +471,13 @@ run (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 }
 
 
+/**
+ * Main function of gnunet-peerinfo-gtk.
+ *
+ * @param argc number of arguments
+ * @param argv arguments
+ * @return 0 on success
+ */
 int
 main (int argc, char *const *argv)
 {