fixed FIXMEs
authorPhilipp Tölke <toelke@in.tum.de>
Mon, 14 Mar 2011 15:16:28 +0000 (15:16 +0000)
committerPhilipp Tölke <toelke@in.tum.de>
Mon, 14 Mar 2011 15:16:28 +0000 (15:16 +0000)
src/vpn/gnunet-daemon-exit.c
src/vpn/gnunet-service-dns.c

index 03e9831c8936e097f284a4d1e46c9b3536f093a1..1c53a83d184043c00c40a8fc9965e06d8871a7a2 100644 (file)
@@ -292,21 +292,23 @@ read_service_conf (void *cls, const char *section, const char *option,
        {     
          if (NULL == (hostname = strstr (redirect, ":")))
            {
-             // FIXME: bitch
+              GNUNET_log(GNUNET_ERROR_TYPE_WARNING, "Warning: option %s is not formatted correctly!\n", redirect);
              continue;
            }
          hostname[0] = '\0';
          hostname++;
          if (NULL == (hostport = strstr (hostname, ":")))
            {
-             // FIXME: bitch
+              GNUNET_log(GNUNET_ERROR_TYPE_WARNING, "Warning: option %s is not formatted correctly!\n", redirect);
              continue;
            }
          hostport[0] = '\0';
          hostport++;
          
           int local_port = atoi (redirect);
-          GNUNET_assert ((local_port > 0) && (local_port < 65536)); // FIXME: don't crash!!!
+          if (!((local_port > 0) && (local_port < 65536)))
+            GNUNET_log(GNUNET_ERROR_TYPE_WARNING, "Warning: %s is not a correct port.", redirect);
+
           *desc = local_port;
 
           GNUNET_CRYPTO_hash (desc, sizeof (GNUNET_HashCode) + 2, &hash);
@@ -642,7 +644,7 @@ run (void *cls,
   };
   mesh_handle = GNUNET_MESH_connect(cfg_,
                                    NULL,
-                                   NULL, /* FIXME */
+                                   NULL,
                                    handlers);
 
   cfg = cfg_;
index 8286b7abda62b7c947f3cd4d4991e0d310da9899..40c3d805297f14f7bdf546f8ee82fb4ad44c7e0e 100644 (file)
@@ -595,12 +595,13 @@ get_port_from_redirects (const char *udp_redirects, const char *tcp_redirects)
         {
           if (NULL == (hostname = strstr (redirect, ":")))
             {
-              // FIXME: bitch
+              GNUNET_log(GNUNET_ERROR_TYPE_WARNING, "Warning: option %s is not formatted correctly!\n", redirect);
               continue;
             }
           hostname[0] = '\0';
           local_port = atoi (redirect);
-          GNUNET_assert ((local_port > 0) && (local_port < 65536)); // FIXME: don't crash!!!
+          if (!((local_port > 0) && (local_port < 65536)))
+            GNUNET_log(GNUNET_ERROR_TYPE_WARNING, "Warning: %s is not a correct port.", redirect);
 
           ret |= (0xFFFF & htons(local_port));
           ret <<= 16;
@@ -623,12 +624,13 @@ get_port_from_redirects (const char *udp_redirects, const char *tcp_redirects)
         {
           if (NULL == (hostname = strstr (redirect, ":")))
             {
-              // FIXME: bitch
+              GNUNET_log(GNUNET_ERROR_TYPE_WARNING, "Warning: option %s is not formatted correctly!\n", redirect);
               continue;
             }
           hostname[0] = '\0';
           local_port = atoi (redirect);
-          GNUNET_assert ((local_port > 0) && (local_port < 65536)); // FIXME: don't crash!!!
+          if (!((local_port > 0) && (local_port < 65536)))
+            GNUNET_log(GNUNET_ERROR_TYPE_WARNING, "Warning: %s is not a correct port.", redirect);
 
           ret |= (0xFFFF & htons(local_port));
           ret <<= 16;