Have "get_positive_int()" really check for positive integers, not just
authorguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 24 Feb 2002 06:01:03 +0000 (06:01 +0000)
committerguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 24 Feb 2002 06:01:03 +0000 (06:01 +0000)
non-negative integers.

Get rid of unused "get_positive_int()" routine in "gtk/capture_dlg.c".

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4796 f5534014-38df-0310-8fa8-9805f1628bb7

gtk/capture_dlg.c
gtk/main.c
tethereal.c

index e0ae080a1a8f4959a35fcbb26a5f1b8a6db884d4..c5aa1eb0b18558aa4351c56a0b06c82e0909ff62 100644 (file)
@@ -1,7 +1,7 @@
 /* capture_dlg.c
  * Routines for packet capture windows
  *
- * $Id: capture_dlg.c,v 1.60 2002/02/24 03:33:05 guy Exp $
+ * $Id: capture_dlg.c,v 1.61 2002/02/24 06:01:03 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -654,35 +654,6 @@ cap_prep_fs_destroy_cb(GtkWidget *win, gpointer data)
   gtk_widget_destroy(GTK_WIDGET(win));
 }
 
-static int
-get_positive_int(const char *string, const char *name)
-{
-  long number;
-  char *p;
-
-  number = strtol(string, &p, 10);
-  /*
-   * XXX - we allow extra stuff after 0, so that we don't have
-   * problems with the "(Infinite)" value.
-   */
-  if (p == string || (*p != '\0' && number != 0)) {
-    simple_dialog(ESD_TYPE_CRIT, NULL,
-        "The specified %s is not a decimal number.", name);
-    return -1;
-  }
-  if (number < 0) {
-    simple_dialog(ESD_TYPE_CRIT, NULL,
-        "The specified %s is a negative number.", name);
-    return -1;
-  }
-  if (number > INT_MAX) {
-    simple_dialog(ESD_TYPE_CRIT, NULL,
-        "The specified %s is too large (greater than %d).", name, INT_MAX);
-    return -1;
-  }
-  return number;
-}
-
 static void
 capture_prep_ok_cb(GtkWidget *ok_bt, gpointer parent_w) {
   GtkWidget *if_cb, *snap_cb, *snap_sb, *promisc_cb, *filter_te,
index bfeceb4a1e4fa64df63886bb905fecffc84bb579..fde8e66c427d634bc870d85357888c2bfd50ec62 100644 (file)
@@ -1,6 +1,6 @@
 /* main.c
  *
- * $Id: main.c,v 1.235 2002/02/24 03:33:05 guy Exp $
+ * $Id: main.c,v 1.236 2002/02/24 06:01:03 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -1114,6 +1114,11 @@ get_positive_int(const char *string, const char *name)
            name, string);
     exit(1);
   }
+  if (number == 0) {
+    fprintf(stderr, "ethereal: The specified %s \"%s\" is zero\n",
+            name, string);
+    exit(1);
+  }
   if (number > INT_MAX) {
     fprintf(stderr, "ethereal: The specified %s \"%s\" is too large (greater than %d)\n",
            name, string, INT_MAX);
@@ -1447,11 +1452,6 @@ main(int argc, char *argv[])
 #ifdef HAVE_LIBPCAP
         has_autostop_count = TRUE;
         autostop_count = get_positive_int(optarg, "packet count");
-        if (autostop_count == 0) {
-          fprintf(stderr, "ethereal: The specified packet count \"%s\" is zero\n",
-                  optarg);
-          exit(1);
-        }
 #else
         capture_option_specified = TRUE;
         arg_error = TRUE;
index d7b2a764f8fb15c140440aa5a6d1b5192c4bff91..33d7b82a7e45dbebaeb1da545a5dd5ede6562fe9 100644 (file)
@@ -1,6 +1,6 @@
 /* tethereal.c
  *
- * $Id: tethereal.c,v 1.124 2002/02/24 03:33:04 guy Exp $
+ * $Id: tethereal.c,v 1.125 2002/02/24 06:01:01 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -200,13 +200,18 @@ get_positive_int(const char *string, const char *name)
     exit(1);
   }
   if (number < 0) {
-    fprintf(stderr, "tethereal: The specified %s \"%s\" is a negative number\n",
-           name, string);
+    fprintf(stderr, "tethereal: The specified %s is a negative number\n",
+           name);
+    exit(1);
+  }
+  if (number == 0) {
+    fprintf(stderr, "tethereal: The specified %s is zero\n",
+           name);
     exit(1);
   }
   if (number > INT_MAX) {
-    fprintf(stderr, "tethereal: The specified %s \"%s\" is too large (greater than %d)\n",
-           name, string, INT_MAX);
+    fprintf(stderr, "tethereal: The specified %s is too large (greater than %d)\n",
+           name, INT_MAX);
     exit(1);
   }
   return number;
@@ -445,11 +450,6 @@ main(int argc, char *argv[])
       case 'c':        /* Capture xxx packets */
 #ifdef HAVE_LIBPCAP
         packet_count = get_positive_int(optarg, "packet count");
-        if (packet_count == 0) {
-          fprintf(stderr, "tethereal: The specified packet count \"%s\" is zero\n",
-                  optarg);
-          exit(1);
-        }
 #else
         capture_option_specified = TRUE;
         arg_error = TRUE;