From Peter Valchev: fix editcap to assign the result of "getopt()" to an
authorguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 24 Feb 2002 01:26:45 +0000 (01:26 +0000)
committerguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Sun, 24 Feb 2002 01:26:45 +0000 (01:26 +0000)
"int" and to check "getopt()"s return value with -1 rather than EOF.

Fix other "getopt()" loops to check against -1 as well (EOF is -1 on
most if not all platforms, but the Single UNIX Specification says
"getopt()" returns -1, so we should check against -1, not EOF).

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

AUTHORS
doc/ethereal.pod.template
editcap.c
gtk/main.c
mergecap.c
randpkt.c
tethereal.c

diff --git a/AUTHORS b/AUTHORS
index da3361ef0341786f709430686369803a1ede0b56..a7133e7416342bd78f1f4f6f22ed8f80e24f5706 100644 (file)
--- a/AUTHORS
+++ b/AUTHORS
@@ -1060,6 +1060,11 @@ John Mackenzie <John.A.Mackenzie[AT]t-online.de> {
        Put missing initializations of table entries in "plugins.c"
 }
 
+Peter Valchev <pvalchev[AT]openbsd.org> {
+       Fix editcap to assign the result of "getopt()" to an "int" and
+          to check "getopt()"s return value with -1 rather than EOF
+}
+
 Alain Magloire <alainm[AT]rcsm.ece.mcgill.ca> was kind enough to
 give his permission to use his version of snprintf.c.
 
index 647b21162986b6e086a34aba356280edaf3c50b1..6066511758393418e84a7ef6602b5f6571463f10 100644 (file)
@@ -1387,6 +1387,7 @@ B<http://www.ethereal.com>.
   Adam Sulmicki            <adam[AT]cfar.umd.edu>
   Kari Tiirikainen         <ktiirika[AT]stybba.ntc.nokia.com>
   John Mackenzie           <John.A.Mackenzie[AT]t-online.de>
+  Peter Valchev            <pvalchev[AT]openbsd.org>
 
 Alain Magloire <alainm[AT]rcsm.ece.mcgill.ca> was kind enough to give his
 permission to use his version of snprintf.c.
index 72b986d28c4a7718a73cec7b1c772af696f2a2d2..efd8b120ce9175841f54743fdeadca2701f8d633 100644 (file)
--- a/editcap.c
+++ b/editcap.c
@@ -1,7 +1,7 @@
 /* Edit capture files.  We can delete records, adjust timestamps, or
  * simply convert from one format to another format.
  *
- * $Id: editcap.c,v 1.19 2002/02/08 10:07:33 guy Exp $
+ * $Id: editcap.c,v 1.20 2002/02/24 01:26:42 guy Exp $
  *
  * Originally written by Richard Sharpe.
  * Improved by Guy Harris.
@@ -314,13 +314,13 @@ int main(int argc, char *argv[])
   callback_arg args;
   extern char *optarg;
   extern int optind;
-  char opt;
+  int opt;
   char *p;
   int snapshot_length;
 
   /* Process the options first */
 
-  while ((opt = getopt(argc, argv, "T:F:rvs:t:h")) != EOF) {
+  while ((opt = getopt(argc, argv, "T:F:rvs:t:h")) !=-1) {
 
     switch (opt) {
 
index 9c42a28eb49021792749577aa6902cff7ff7835a..40a181512872085d21a847f49be534b7c8e48cb5 100644 (file)
@@ -1,6 +1,6 @@
 /* main.c
  *
- * $Id: main.c,v 1.233 2002/02/18 01:08:44 guy Exp $
+ * $Id: main.c,v 1.234 2002/02/24 01:26:45 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -1412,7 +1412,7 @@ main(int argc, char *argv[])
 #endif
 
   /* Now get our args */
-  while ((opt = getopt(argc, argv, "a:b:B:c:f:hi:klm:nN:o:pP:Qr:R:Ss:t:T:w:W:vZ:")) !=  EOF) {
+  while ((opt = getopt(argc, argv, "a:b:B:c:f:hi:klm:nN:o:pP:Qr:R:Ss:t:T:w:W:vZ:")) != -1) {
     switch (opt) {
       case 'a':        /* autostop criteria */
 #ifdef HAVE_LIBPCAP
index d5ce6c63d12a9afcea3bd150db033091b5c20371..024cbc74c46ed962960e76771f7a9add2cc845c0 100644 (file)
@@ -1,6 +1,6 @@
 /* Combine two dump files, either by appending or by merging by timestamp
  *
- * $Id: mergecap.c,v 1.6 2002/02/08 10:07:34 guy Exp $
+ * $Id: mergecap.c,v 1.7 2002/02/24 01:26:42 guy Exp $
  *
  * Written by Scott Renfro <scott@renfro.org> based on
  * editcap by Richard Sharpe and Guy Harris
@@ -405,7 +405,7 @@ main(int argc, char *argv[])
   out_file.count      = 1;                 /* frames output */
 
   /* Process the options first */
-  while ((opt = getopt(argc, argv, "hvas:T:F:w:")) != EOF) {
+  while ((opt = getopt(argc, argv, "hvas:T:F:w:")) != -1) {
 
     switch (opt) {
     case 'w':
index 2b1192d7d6586ae3c89d2f50b6aaa9d4209f43f2..da17a8aa4cf77ba5bf39badfe0bf9fd9cc1e4961 100644 (file)
--- a/randpkt.c
+++ b/randpkt.c
@@ -4,7 +4,7 @@
  * Creates random packet traces. Useful for debugging sniffers by testing
  * assumptions about the veracity of the data found in the packet.
  *
- * $Id: randpkt.c,v 1.12 2002/02/14 17:45:07 gram Exp $
+ * $Id: randpkt.c,v 1.13 2002/02/24 01:26:42 guy Exp $
  *
  * Copyright (C) 1999 by Gilbert Ramirez <gram@alumni.rice.edu>
  * 
@@ -304,7 +304,7 @@ main(int argc, char **argv)
        int                     produce_max_bytes = 5000;
        pkt_example             *example;
 
-       while ((opt = getopt(argc, argv, "b:c:t:")) != EOF) {
+       while ((opt = getopt(argc, argv, "b:c:t:")) != -1) {
                switch (opt) {
                        case 'b':       /* max bytes */
                                produce_max_bytes = atoi(optarg);
index 41fa5d0d4a2430801ea6c57b18973b917de2a5ad..65b9d541a89295e022e2fa8f7fadcc51d7694591 100644 (file)
@@ -1,6 +1,6 @@
 /* tethereal.c
  *
- * $Id: tethereal.c,v 1.122 2002/02/18 01:08:38 guy Exp $
+ * $Id: tethereal.c,v 1.123 2002/02/24 01:26:42 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -415,7 +415,7 @@ main(int argc, char *argv[])
 #endif
     
   /* Now get our args */
-  while ((opt = getopt(argc, argv, "a:b:c:Df:F:hi:lnN:o:pr:R:s:t:vw:Vx")) != EOF) {
+  while ((opt = getopt(argc, argv, "a:b:c:Df:F:hi:lnN:o:pr:R:s:t:vw:Vx")) != -1) {
     switch (opt) {
       case 'a':        /* autostop criteria */
 #ifdef HAVE_LIBPCAP