make clang static analysis happy
[oweals/gnunet.git] / src / hostlist / gnunet-daemon-hostlist.c
index cf4be4798f606ef6b1ef63081f1c906d0cee540b..3a1d53983453a92a1239b0ccec68c6ceb41cb154 100644 (file)
  * @file hostlist/gnunet-daemon-hostlist.c
  * @brief code for bootstrapping via hostlist servers
  * @author Christian Grothoff
- *
- * TODO:
- * - implement -a and -e switches (send P2P messages about our hostlist URL,
- *   receive such messages and automatically update our hostlist URL config
- *   value).
  */
 
 #include <stdlib.h>
 #include "platform.h"
 #include "hostlist-client.h"
-#include "hostlist-server.h"
 #include "gnunet_core_service.h"
 #include "gnunet_getopt_lib.h"
 #include "gnunet_protocols.h"
 #include "gnunet_time_lib.h"
 #include "gnunet_util_lib.h"
 
-#define DEBUG_HOSTLIST GNUNET_YES
+#if HAVE_MHD
+
+#include "hostlist-server.h"
 
 /**
  * Set if we are allowed to advertise our hostlist to others.
  */
 static int advertising;
 
+/**
+ * Set if the user wants us to run a hostlist server.
+ */
+static int provide_hostlist;
+
+/**
+ * Handle to hostlist server's connect handler
+ */
+static GNUNET_CORE_ConnectEventHandler server_ch = NULL;
+
+/**
+ * Handle to hostlist server's disconnect handler
+ */
+static GNUNET_CORE_DisconnectEventHandler server_dh = NULL;
+
+#endif
+
 /**
  * Set if we are allowed to learn about peers by accessing
  * hostlist servers.
@@ -61,11 +74,6 @@ static int bootstrapping;
  */
 static int learning;
 
-/**
- * Set if the user wants us to run a hostlist server.
- */
-static int provide_hostlist;
-
 /**
  * Statistics handle.
  */
@@ -74,52 +82,71 @@ static struct GNUNET_STATISTICS_Handle *stats;
 /**
  * Handle to the core service (NULL until we've connected to it).
  */
-struct GNUNET_CORE_Handle *core;
+static struct GNUNET_CORE_Handle *core;
 
 /**
  * Handle to the hostlist client's advertisement handler
  */
-GNUNET_CORE_MessageCallback client_adv_handler = NULL;
+static GNUNET_CORE_MessageCallback client_adv_handler = NULL;
 
 /**
  * Handle to hostlist client's connect handler
  */
-GNUNET_CORE_ConnectEventHandler client_ch = NULL;
+static GNUNET_CORE_ConnectEventHandler client_ch = NULL;
 
 /**
  * Handle to hostlist client's disconnect handler
  */
-GNUNET_CORE_DisconnectEventHandler client_dh = NULL;
-
-/**
- * Handle to hostlist server's connect handler
- */
-GNUNET_CORE_ConnectEventHandler server_ch = NULL;
-
-/**
- * Handle to hostlist server's disconnect handler
- */
-GNUNET_CORE_DisconnectEventHandler server_dh = NULL;
+static GNUNET_CORE_DisconnectEventHandler client_dh = NULL;
 
 /**
  * gnunet-daemon-hostlist command line options.
  */
 static struct GNUNET_GETOPT_CommandLineOption options[] = {
+#if HAVE_MHD
   { 'a', "advertise", NULL, 
     gettext_noop ("advertise our hostlist to other peers"),
     GNUNET_NO, &GNUNET_GETOPT_set_one, &advertising },
+#endif
   { 'b', "bootstrap", NULL, 
     gettext_noop ("bootstrap using hostlists (it is highly recommended that you always use this option)"),
     GNUNET_NO, &GNUNET_GETOPT_set_one, &bootstrapping },
   { 'e', "enable-learning", NULL,
     gettext_noop ("enable learning about hostlist servers from other peers"),
     GNUNET_NO, &GNUNET_GETOPT_set_one, &learning},
+#if HAVE_MHD
   { 'p', "provide-hostlist", NULL, 
     gettext_noop ("provide a hostlist server"),
     GNUNET_NO, &GNUNET_GETOPT_set_one, &provide_hostlist},
+#endif
   GNUNET_GETOPT_OPTION_END
 };
 
+/**
+ * A HOSTLIST_ADV message is used to exchange information about
+ * hostlist advertisements.  This struct is always
+ * followed by the actual url under which the hostlist can be obtained:
+ *
+ * 1) transport-name (0-terminated)
+ * 2) address-length (uint32_t, network byte order; possibly
+ *    unaligned!)
+ * 3) address expiration (GNUNET_TIME_AbsoluteNBO); possibly
+ *    unaligned!)
+ * 4) address (address-length bytes; possibly unaligned!)
+ */
+struct GNUNET_HOSTLIST_ADV_Message
+{
+  /**
+   * Type will be GNUNET_MESSAGE_TYPE_HOSTLIST_ADVERTISEMENT.
+   */
+  struct GNUNET_MessageHeader header;
+
+  /**
+   * Always zero (for alignment).
+   */
+  uint32_t reserved GNUNET_PACKED;
+};
+
 
 static void
 core_init (void *cls,
@@ -130,10 +157,7 @@ core_init (void *cls,
           GNUNET_CRYPTO_RsaPublicKeyBinaryEncoded *
           publicKey)
 {
-  if (advertising && (NULL != server))
-    {    
-      /* FIXME: provide "server" to 'hostlist' module */
-    }
+ /* Nothing to do here */
 }
 
 /**
@@ -145,14 +169,11 @@ static int advertisement_handler (void *cls,
                              struct GNUNET_TIME_Relative latency,
                              uint32_t distance)
 {
-  if (advertising && (NULL != client_adv_handler))
-    {
-        (*client_adv_handler) (cls, peer, message, latency, distance);
-        return GNUNET_YES;
-    }
-  return GNUNET_NO;
+  GNUNET_assert (NULL != client_adv_handler);
+  return (*client_adv_handler) (cls, peer, message, latency, distance);
 }
 
+
 /**
  * Method called whenever a given peer connects.  Wrapper to call both client's and server's functions
  *
@@ -172,8 +193,10 @@ connect_handler (void *cls,
               "A new peer connected, notifying client and server\n");
   if ( NULL != client_ch)
     (*client_ch) (cls, peer, latency, distance);
+#if HAVE_MHD
   if ( NULL != server_ch)
     (*server_ch) (cls, peer, latency, distance);
+#endif
 }
 
 /**
@@ -181,8 +204,6 @@ connect_handler (void *cls,
  *
  * @param cls closure
  * @param peer peer identity this notification is about
- * @param latency reported latency of the connection with 'other'
- * @param distance reported distance (DV) to 'other'
  */
 static void
 disconnect_handler (void *cls,
@@ -193,10 +214,11 @@ disconnect_handler (void *cls,
   /* call hostlist client disconnect handler*/
   if ( NULL != client_dh)
     (*client_dh) (cls, peer);
-
+#if HAVE_MHD
   /* call hostlist server disconnect handler*/
   if ( NULL != server_dh)
     (*server_dh) (cls, peer);
+#endif
 }
 
 /**
@@ -212,10 +234,12 @@ cleaning_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
     {
       GNUNET_HOSTLIST_client_stop ();
     }
+#if HAVE_MHD
   if (provide_hostlist)
     {      
       GNUNET_HOSTLIST_server_stop ();
     }
+#endif
   if (core != NULL)
     {
       GNUNET_CORE_disconnect (core);
@@ -230,14 +254,22 @@ cleaning_task (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
 }
 
 /**
- * List of handlers for the messages understood by this
- * service.
+ * List of handlers if we are learning.
  */
-static struct GNUNET_CORE_MessageHandler handlers[] = {
-    { &advertisement_handler, GNUNET_MESSAGE_TYPE_HOSTLIST_ADVERTISEMENT, 0},
+static struct GNUNET_CORE_MessageHandler learn_handlers[] = {
+  { &advertisement_handler, GNUNET_MESSAGE_TYPE_HOSTLIST_ADVERTISEMENT, 0},
+  { NULL, 0, 0 }
+};
+
+
+/**
+ * List of handlers if we are not learning.
+ */
+static struct GNUNET_CORE_MessageHandler no_learn_handlers[] = {
     { NULL, 0, 0 }
 };
 
+
 /**
  * Main function that will be run.
  *
@@ -255,37 +287,38 @@ run (void *cls,
      const struct GNUNET_CONFIGURATION_Handle * cfg)
 {
   if ( (! bootstrapping) &&
-       (! learning) &&
-       (! provide_hostlist) )
+       (! learning) 
+#if HAVE_MHD
+       && (! provide_hostlist) 
+#endif
+       )
     {
       GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
                  _("None of the functions for the hostlist daemon were enabled.  I have no reason to run!\n"));
       return;
     }
   stats = GNUNET_STATISTICS_create (sched, "hostlist", cfg);
+
+  core = GNUNET_CORE_connect (sched, cfg,
+                             GNUNET_TIME_UNIT_FOREVER_REL,
+                             NULL,
+                             &core_init,
+                             &connect_handler, &disconnect_handler,
+                             NULL, GNUNET_NO,
+                             NULL, GNUNET_NO,
+                             learning? learn_handlers : no_learn_handlers);
+
   if (bootstrapping)
     {
       GNUNET_HOSTLIST_client_start (cfg, sched, stats,
-                                   &client_ch, &client_dh, &client_adv_handler);
+                                   &client_ch, &client_dh, &client_adv_handler, learning);
     }
+#if HAVE_MHD
   if (provide_hostlist)
     {      
-      GNUNET_HOSTLIST_server_start (cfg, sched, stats, &server_ch, &server_dh);
-    }
-  if (learning)
-    {
-
+      GNUNET_HOSTLIST_server_start (cfg, sched, stats, core, &server_ch, &server_dh, advertising );
     }
-
-  core = GNUNET_CORE_connect (sched, cfg,
-                            GNUNET_TIME_UNIT_FOREVER_REL,
-                            NULL,
-                            &core_init,
-                            NULL, &connect_handler, &disconnect_handler,
-                            NULL, GNUNET_NO,
-                            NULL, GNUNET_NO,
-                            handlers);
-
+#endif
   GNUNET_SCHEDULER_add_delayed (sched,
                                 GNUNET_TIME_UNIT_FOREVER_REL,
                                 &cleaning_task, NULL);