minor fixes
authorChristian Grothoff <christian@grothoff.org>
Fri, 11 Jun 2010 22:33:24 +0000 (22:33 +0000)
committerChristian Grothoff <christian@grothoff.org>
Fri, 11 Jun 2010 22:33:24 +0000 (22:33 +0000)
src/hostlist/hostlist-server.c
src/include/platform.h
src/peerinfo/gnunet-service-peerinfo.c
src/testing/testing_group.c
src/util/strings.c

index 055c90bab439c9f82ee8e16520d9efa72510fc9b..8d4d38d0912b410b7f7d5fafd88b94ccda6c5865 100644 (file)
@@ -597,11 +597,19 @@ GNUNET_HOSTLIST_server_start (const struct GNUNET_CONFIGURATION_Handle *c,
                  _("Could not access PEERINFO service.  Exiting.\n"));     
       return GNUNET_SYSERR;
     }
-  if (-1 == GNUNET_CONFIGURATION_get_value_number (cfg,
-                                                  "HOSTLIST",
-                                                  "HTTPPORT", 
-                                                  &port))
+  if (GNUNET_OK != GNUNET_CONFIGURATION_get_value_number (cfg,
+                                                         "HOSTLIST",
+                                                         "HTTPPORT", 
+                                                         &port))
     return GNUNET_SYSERR;
+  if ( (port == 0) ||
+       (port > UINT16_MAX) )
+    {
+      GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                 _("Invalid port number %llu.  Exiting.\n"),
+                 port);            
+      return GNUNET_SYSERR;
+    }
 
   if ( GNUNET_SYSERR  == GNUNET_CONFIGURATION_get_value_string (cfg,
                                                    "HOSTLIST",
index fb9c992ec80aea58336a21d568ced21cacbadffd..7a974b48d7532a80dcd4a0e3a47ccf5d7ed23331 100644 (file)
@@ -214,6 +214,10 @@ long long atoll (const char *nptr);
 #include "langinfo.h"
 #endif
 
+#ifndef SIZE_MAX
+#define SIZE_MAX ((size_t)(-1))
+#endif
+
 #ifndef O_LARGEFILE
 #define O_LARGEFILE 0
 #endif
index e8ac5ef4aee17eb691f90066706d8c9116ba05a1..7c1f53d4bd4ea05556d6203a137e43f3fdc67899 100644 (file)
@@ -329,8 +329,8 @@ change_host_trust (const struct GNUNET_PeerIdentity *hostId, int value)
     {
       if (host->trust + value < host->trust)
         {
-          value = ((uint32_t) - 1) - host->trust;
-          host->trust = (uint32_t) - 1; /* maximized */
+          value = UINT32_MAX - host->trust;
+          host->trust = UINT32_MAX;
         }
       else
         host->trust += value;
index 4f369812de972afb525ff15c631b34818a7b12cb..f4057e6d13fa1251ab30d44a032cd530adcd1a06 100644 (file)
@@ -1488,7 +1488,7 @@ create_and_copy_blacklist_files (struct GNUNET_TESTING_PeerGroup *pg, char *tran
         }
       }
 
-      GNUNET_free_non_null(temp_transports);
+      GNUNET_free (temp_transports);
       fclose(temp_file_handle);
 
       if (GNUNET_OK !=
index c868d1a9765c1567a53d9e2cefcbaeb5eff9108c..1f7f45d5192504cb27e3b0812d9d3d78da24673a 100644 (file)
@@ -204,7 +204,7 @@ GNUNET_STRINGS_to_utf8 (const char *input, size_t len, const char *charset)
 #else
              (char **) &input,
 #endif
-             &len, &itmp, &finSize) == (size_t) - 1)
+             &len, &itmp, &finSize) == SIZE_MAX)
     {
       GNUNET_log_strerror (GNUNET_ERROR_TYPE_WARNING, "iconv");
       iconv_close (cd);