use more proper way to parse cli options
authorJulius Bünger <buenger@mytum.de>
Mon, 25 Jun 2018 11:43:16 +0000 (13:43 +0200)
committerJulius Bünger <buenger@mytum.de>
Mon, 25 Jun 2018 11:43:55 +0000 (13:43 +0200)
src/rps/gnunet-rps.c

index e09277589cb06dbed21d365c01a2f465ddebeb29..739f71dac5f531a772e91a3531ea6495687d204e 100644 (file)
@@ -155,7 +155,11 @@ run (void *cls,
       (!view_update))
   { /* Request n PeerIDs */
     /* If number was specified use it, else request single peer. */
-    num_peers = (NULL == args[0]) ? 1 : atoi (args[0]);
+    if (NULL == args[0] ||
+        0 == sscanf (args[0], "%lu", &num_peers))
+    {
+      num_peers = 1;
+    }
     GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
         "Requesting %" PRIu64 " PeerIDs\n", num_peers);
     req_handle = GNUNET_RPS_request_peers (rps_handle, num_peers, reply_handle, NULL);
@@ -163,7 +167,11 @@ run (void *cls,
   } else if (view_update)
   {
     /* Get updates of view */
-    num_view_updates = (NULL == args[0]) ? 0 : atoi (args[0]);
+    if (NULL == args[0] ||
+        0 == sscanf (args[0], "%lu", &num_view_updates))
+    {
+      num_view_updates = 0;
+    }
     GNUNET_RPS_view_request (rps_handle, num_view_updates, view_update_handle, NULL);
     if (0 != num_view_updates)
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,