From Joerg Mayer:
authorguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 13 Jul 2001 00:55:58 +0000 (00:55 +0000)
committerguy <guy@f5534014-38df-0310-8fa8-9805f1628bb7>
Fri, 13 Jul 2001 00:55:58 +0000 (00:55 +0000)
 * gcc 3.0 warning fixes:
  - text2pcap.c: The number of characters to scan should probably not be 0
  - wiretap/csids.c: using preincrement on a variable used on both
    sides of an assignment might be undefined by the C99(?) standard
 * turn on additional warnings for epan and wiretap too
  - epan/configure.in
  - wiretap/configure.in
 * Fix some warnings (missing includes, signed/unsigned, missing
    initializers) found by turning on the warnings
  - all other files :-)

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

19 files changed:
epan/configure.in
epan/dfilter/dfilter.c
epan/ftypes/ftype-double.c
epan/ftypes/ftype-integer.c
epan/ftypes/ftype-ipv4.c
epan/ftypes/ftype-none.c
epan/ftypes/ftype-time.c
epan/proto.c
text2pcap.c
wiretap/ascend-grammar.y
wiretap/ascend.c
wiretap/configure.in
wiretap/csids.c
wiretap/etherpeek.c
wiretap/file.c
wiretap/i4b_trace.h
wiretap/libpcap.c
wiretap/netmon.c
wiretap/wtap.c

index e24d87d3cb6c79b7d3fe48eac5161fbd8f6b5d2f..05b98cde91f961453b78c4b40707aaa877d6b7f4 100644 (file)
@@ -1,4 +1,4 @@
-# $Id: configure.in,v 1.8 2001/05/16 16:17:52 jfoster Exp $
+# $Id: configure.in,v 1.9 2001/07/13 00:55:53 guy Exp $
 dnl
 dnl Process this file with autoconf 2.13 or later to produce a
 dnl configure script; 2.12 doesn't generate a "configure" script that
@@ -29,7 +29,7 @@ AC_SUBST(FLEX_PATH)
 # If we're running gcc, add '-Wall' to CFLAGS.
 AC_MSG_CHECKING(to see if we can add '-Wall' to CFLAGS)
 if test x$GCC != x ; then
-  CFLAGS="-Wall $CFLAGS"
+  CFLAGS="-Wall -W -Wno-unused $CFLAGS"
   AC_MSG_RESULT(yes)
 else
   AC_MSG_RESULT(no)
index ac45f2eb4d1d27759aacf85e30dc82c57b1120d8..0380f15662e948f5d203a4a76baf36235931ca63 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: dfilter.c,v 1.4 2001/02/13 18:34:50 guy Exp $
+ * $Id: dfilter.c,v 1.5 2001/07/13 00:55:54 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -26,6 +26,7 @@
 #endif
 
 #include <stdio.h>
+#include <string.h>
 
 #ifdef NEED_SNPRINTF_H
 #include "snprintf.h"
@@ -211,7 +212,7 @@ dfilter_new(void)
 static void
 free_insns(GPtrArray *insns)
 {
-       int             i;
+       unsigned int    i;
        dfvm_insn_t     *insn;
 
        for (i = 0; i < insns->len; i++) {
index ba19a8d8e7b558eda1b84aa504fd1ea2a9185b5a..8fa42e0032a855b6c50741142cc2137ca546b0f1 100644 (file)
@@ -1,6 +1,6 @@
 /*
  *
- * $Id: ftype-double.c,v 1.3 2001/03/02 17:17:56 gram Exp $
+ * $Id: ftype-double.c,v 1.4 2001/07/13 00:55:56 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -140,6 +140,9 @@ ftype_register_double(void)
                cmp_ge,
                cmp_lt,
                cmp_le,
+
+               NULL,
+               NULL,
        };
 
        ftype_register(FT_DOUBLE, &double_type);
index c0da733cea96fe72d90d378e16676339dad3115f..fcfd996e97c9e7417c24416ddf1f60c20ab1e59d 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: ftype-integer.c,v 1.5 2001/03/03 00:33:24 guy Exp $
+ * $Id: ftype-integer.c,v 1.6 2001/07/13 00:55:56 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -232,6 +232,9 @@ ftype_register_integers(void)
                u_cmp_ge,
                u_cmp_lt,
                u_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t uint16_type = {
                "FT_UINT16",
@@ -255,6 +258,9 @@ ftype_register_integers(void)
                u_cmp_ge,
                u_cmp_lt,
                u_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t uint24_type = {
                "FT_UINT24",
@@ -278,6 +284,9 @@ ftype_register_integers(void)
                u_cmp_ge,
                u_cmp_lt,
                u_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t uint32_type = {
                "FT_UINT32",
@@ -301,6 +310,9 @@ ftype_register_integers(void)
                u_cmp_ge,
                u_cmp_lt,
                u_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t int8_type = {
                "FT_INT8",
@@ -324,6 +336,9 @@ ftype_register_integers(void)
                s_cmp_ge,
                s_cmp_lt,
                s_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t int16_type = {
                "FT_INT16",
@@ -347,6 +362,9 @@ ftype_register_integers(void)
                s_cmp_ge,
                s_cmp_lt,
                s_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t int24_type = {
                "FT_INT24",
@@ -370,6 +388,9 @@ ftype_register_integers(void)
                s_cmp_ge,
                s_cmp_lt,
                s_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t int32_type = {
                "FT_INT32",
@@ -393,6 +414,9 @@ ftype_register_integers(void)
                s_cmp_ge,
                s_cmp_lt,
                s_cmp_le,
+
+               NULL,
+               NULL,
        };
        static ftype_t boolean_type = {
                "FT_BOOLEAN",
@@ -416,6 +440,9 @@ ftype_register_integers(void)
                NULL,
                NULL,
                NULL,
+
+               NULL,
+               NULL,
        };
 
        static ftype_t ipxnet_type = {
@@ -440,6 +467,9 @@ ftype_register_integers(void)
                u_cmp_ge,
                u_cmp_lt,
                u_cmp_le,
+
+               NULL,
+               NULL,
        };
 
 
index 03e8d3a92cda7e9d93b099314c7ed4c5e3c4f348..0037923a922db7dee917425e816177e5db7e28f3 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: ftype-ipv4.c,v 1.5 2001/06/23 19:10:27 guy Exp $
+ * $Id: ftype-ipv4.c,v 1.6 2001/07/13 00:55:56 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -180,6 +180,9 @@ ftype_register_ipv4(void)
                cmp_ge,
                cmp_lt,
                cmp_le,
+
+               NULL,
+               NULL,
        };
 
        ftype_register(FT_IPv4, &ipv4_type);
index f19b6e2714c69abaed5333c442f92f3014f5c2a6..09884aeb4a91fe620fc3da73c214a475fb6e9946 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: ftype-none.c,v 1.2 2001/02/01 20:31:21 gram Exp $
+ * $Id: ftype-none.c,v 1.3 2001/07/13 00:55:56 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@zing.org>
@@ -36,7 +36,27 @@ ftype_register_none(void)
                "FT_NONE",
                "label",
                0,
-       };
+               NULL,
+               NULL,
+               NULL,
+
+               NULL,
+               NULL,
+               NULL,
+
+               NULL,
+               NULL,
+               NULL,
 
+               NULL,
+               NULL,
+               NULL,
+               NULL,
+               NULL,
+               NULL,
+
+               NULL,
+               NULL,
+       };
        ftype_register(FT_NONE, &none_type);
 }
index bfaa18a0f5bb86183594c3d94f0ac0e2a7e99fb0..aeab4c1c8ee6419063322114e07eda824e752a78 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: ftype-time.c,v 1.9 2001/06/19 23:08:57 guy Exp $
+ * $Id: ftype-time.c,v 1.10 2001/07/13 00:55:56 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -25,6 +25,7 @@
 #endif
 
 #include <ctype.h>
+#include <string.h>
 
 /*
  * Just make sure we include the prototype for strptime as well
index f6d5f1464ca5c6d4a3c63dc2672d5519c9f5f0ed..35de4bde33f630eab9c64f7cf65538c220847b65 100644 (file)
@@ -1,7 +1,7 @@
 /* proto.c
  * Routines for protocol tree
  *
- * $Id: proto.c,v 1.29 2001/06/05 07:38:35 guy Exp $
+ * $Id: proto.c,v 1.30 2001/07/13 00:55:53 guy Exp $
  *
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -168,7 +168,7 @@ proto_init(const char *plugin_dir,void (register_all_protocols)(void),
        static hf_register_info hf[] = {
                { &hf_text_only,
                { "",   "", FT_NONE, BASE_NONE, NULL, 0x0,
-                       NULL }},
+                       NULL, HFILL }},
        };
 
        if (gmc_hfinfo)
index 17508a172e23f5737730cf97f1254ce3af432a65..dd089a39b3a27874371a2b03bf0e04623d662f73 100644 (file)
@@ -6,7 +6,7 @@
  *
  * (c) Copyright 2001 Ashok Narayanan <ashokn@cisco.com>
  *
- * $Id: text2pcap.c,v 1.2 2001/05/21 03:17:14 guy Exp $
+ * $Id: text2pcap.c,v 1.3 2001/07/13 00:55:52 guy Exp $
  * 
  * Ethereal - Network traffic analyzer
  * By Gerald Combs <gerald@ethereal.com>
@@ -622,7 +622,7 @@ parse_options (int argc, char *argv[])
             break;
         case 'e':
             hdr_ethernet = TRUE;
-            if (!optarg || sscanf(optarg, "%0lx", &hdr_ethernet_proto) < 1) {
+            if (!optarg || sscanf(optarg, "%lx", &hdr_ethernet_proto) < 1) {
                 fprintf(stderr, "Bad argument for '-e': %s\n",
                         optarg ? optarg : "");
                 help(argv[0]);
index 08d989ee11829278f1dc2e55ce30f0b0bc70003e..4577c935dd1ec59e26014baa48ca55e7070a3621 100644 (file)
@@ -1,7 +1,7 @@
 %{
 /* ascend-grammar.y
  *
- * $Id: ascend-grammar.y,v 1.18 2001/04/09 03:32:34 gerald Exp $
+ * $Id: ascend-grammar.y,v 1.19 2001/07/13 00:55:57 guy Exp $
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@xiexie.org>
@@ -86,7 +86,7 @@ extern int at_eof;
 int yyparse(void);
 void yyerror(char *);
 
-int bcur = 0, bcount;
+unsigned int bcur = 0, bcount;
 guint32 start_time, secs, usecs, caplen, wirelen;
 ascend_pkthdr *header;
 struct ascend_phdr *pseudo_header;
index 7155815403792a0f9498e45fa8b0db7991f74f34..04a34783cf18903fba8dc692640acd7cf771f4ce 100644 (file)
@@ -1,6 +1,6 @@
 /* ascend.c
  *
- * $Id: ascend.c,v 1.23 2001/03/10 06:33:57 guy Exp $
+ * $Id: ascend.c,v 1.24 2001/07/13 00:55:57 guy Exp $
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@xiexie.org>
@@ -116,7 +116,7 @@ static void ascend_close(wtap *wth);
 static int ascend_seek(wtap *wth, int max_seek)
 {
   int byte, bytes_read = 0, date_off = -1, cur_off, packet_off;
-  int x_level = 0, r_level = 0, w1_level = 0, w2_level = 0;
+  unsigned int r_level = 0, x_level = 0, w1_level = 0, w2_level = 0;
 
   while (((byte = file_getc(wth->fh)) != EOF) && bytes_read < max_seek) {
     if (byte == ascend_xmagic[x_level]) {
index b9d42b581b7e5aa3f204f2f5ebdeae9cf714e3d7..88c23375c8892340015916c422f071037718055b 100644 (file)
@@ -1,4 +1,4 @@
-# $Id: configure.in,v 1.27 2001/06/27 07:47:50 guy Exp $
+# $Id: configure.in,v 1.28 2001/07/13 00:55:57 guy Exp $
 dnl
 dnl Process this file with autoconf 2.13 or later to produce a
 dnl configure script; 2.12 doesn't generate a "configure" script that
@@ -29,7 +29,7 @@ AC_SUBST(FLEX_PATH)
 # If we're running gcc, add '-Wall' to CFLAGS.
 AC_MSG_CHECKING(to see if we can add '-Wall' to CFLAGS)
 if test x$GCC != x ; then
-  CFLAGS="-Wall $CFLAGS"
+  CFLAGS="-Wall -W -Wno-unused $CFLAGS"
   AC_MSG_RESULT(yes)
 else
   AC_MSG_RESULT(no)
index d50ae18c3f7a82530fa6a94219483bd8e17e693a..7196b0ea92992fba7845d36cf3194dff7a7ab4f4 100644 (file)
@@ -1,6 +1,6 @@
 /* csids.c
  *
- * $Id: csids.c,v 1.6 2001/03/10 06:33:57 guy Exp $
+ * $Id: csids.c,v 1.7 2001/07/13 00:55:58 guy Exp $
  *
  * Copyright (c) 2000 by Mike Hall <mlh@io.com>
  * Copyright (c) 2000 by Cisco Systems
@@ -180,9 +180,12 @@ static gboolean csids_read(wtap *wth, int *err, int *data_offset)
 
   if( wth->capture.csids->byteswapped == TRUE ) {
     guint16* swap = (guint16*)buf;
-    *(++swap) = BSWAP16(*swap); /* the ip len */
-    *(++swap) = BSWAP16(*swap); /* ip id */
-    *(++swap) = BSWAP16(*swap); /* ip flags and fragoff */
+    swap++;
+    *(swap) = BSWAP16(*swap); /* the ip len */
+    swap++;
+    *(swap) = BSWAP16(*swap); /* ip id */
+    swap++;
+    *(swap) = BSWAP16(*swap); /* ip flags and fragoff */
   }
 
   return TRUE;
@@ -228,9 +231,12 @@ csids_seek_read (wtap *wth,
 
   if( wth->capture.csids->byteswapped == TRUE ) {
     guint16* swap = (guint16*)pd;
-    *(++swap) = BSWAP16(*swap); /* the ip len */
-    *(++swap) = BSWAP16(*swap); /* ip id */
-    *(++swap) = BSWAP16(*swap); /* ip flags and fragoff */
+    swap++;
+    *(swap) = BSWAP16(*swap); /* the ip len */
+    swap++;
+    *(swap) = BSWAP16(*swap); /* ip id */
+    swap++;
+    *(swap) = BSWAP16(*swap); /* ip flags and fragoff */
   }
   
   return 0;
index 5a6181b55fe97ae05e13fbb16c839a358faaaab5..32102eba20df4d5f28c6d5410fe9450234544505 100644 (file)
@@ -2,7 +2,7 @@
  * Routines for opening etherpeek files
  * Copyright (c) 2001, Daniel Thompson <d.thompson@gmx.net>
  *
- * $Id: etherpeek.c,v 1.3 2001/03/10 06:33:57 guy Exp $
+ * $Id: etherpeek.c,v 1.4 2001/07/13 00:55:58 guy Exp $
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@xiexie.org>
@@ -210,7 +210,7 @@ static gboolean etherpeek_read_m7(wtap *wth, int *err, int *data_offset)
 {
        etherpeek_m7_packet_t ep_pkt;
        double  t;
-       int i;
+       unsigned int i;
 
        g_assert(sizeof(ep_pkt) == ETHERPEEK_M7_PKT_SIZE);
        wtap_file_read_expected_bytes(&ep_pkt, sizeof(ep_pkt), wth->fh, err);
@@ -265,7 +265,7 @@ static gboolean etherpeek_read_m7(wtap *wth, int *err, int *data_offset)
 static gboolean etherpeek_read_m56(wtap *wth, int *err, int *data_offset)
 {
        etherpeek_m56_packet_t ep_pkt;
-       int i;
+       unsigned int i;
 
        g_assert(sizeof(ep_pkt) == ETHERPEEK_M56_PKT_SIZE);
        wtap_file_read_expected_bytes(&ep_pkt, sizeof(ep_pkt), wth->fh, err);
index 6eba00d99398347bd0c85d0caa4205caa69c0878..62b5dcdbeaa68409daddaca8626a4f15d0bff974 100644 (file)
@@ -1,6 +1,6 @@
 /* file.c
  *
- * $Id: file.c,v 1.66 2001/03/10 06:33:57 guy Exp $
+ * $Id: file.c,v 1.67 2001/07/13 00:55:58 guy Exp $
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@xiexie.org>
@@ -149,7 +149,7 @@ wtap* wtap_open_offline(const char *filename, int *err, gboolean do_random)
 {
        struct stat statb;
        wtap    *wth;
-       int     i;
+       unsigned int    i;
 
        /* First, make sure the file is valid */
        if (stat(filename, &statb) < 0) {
@@ -252,7 +252,7 @@ success:
 }
 
 /* Table of the file types we know about. */
-const static struct file_type_info {
+static const struct file_type_info {
        const char *name;
        const char *short_name;
        int     (*can_write_encap)(int, int);
index 59684e1dd2df4686c2892cdd976bdaf20786a2f0..66ef59d87689d035649ca3faaebe4a6432598f72 100644 (file)
@@ -27,7 +27,7 @@
  *     i4b_trace.h - header file for trace data read device
  *     ----------------------------------------------------
  *
- *     $Id: i4b_trace.h,v 1.1 1999/12/12 22:40:09 gram Exp $ 
+ *     $Id: i4b_trace.h,v 1.2 2001/07/13 00:55:58 guy Exp $ 
  *
  *      last edit-date: [Sun Feb 14 10:39:26 1999]
  *
@@ -40,7 +40,7 @@
  *     structure of the header at the beginning of every trace mbuf
  *---------------------------------------------------------------------------*/
 typedef struct {
-       int length;             /* length of the following mbuf         */
+       unsigned int length;    /* length of the following mbuf         */
        int unit;               /* controller unit number               */
        int type;               /* type of channel                      */
 #define TRC_CH_I       0               /* Layer 1 INFO's               */
index b85d518e382f76de476f67f472b534fcc90e209c..294bb95d661778cc482ea496c5156f3211e618ff 100644 (file)
@@ -1,6 +1,6 @@
 /* libpcap.c
  *
- * $Id: libpcap.c,v 1.48 2001/03/15 09:11:03 guy Exp $
+ * $Id: libpcap.c,v 1.49 2001/07/13 00:55:58 guy Exp $
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@xiexie.org>
@@ -867,7 +867,7 @@ libpcap_close(wtap *wth)
 
 int wtap_pcap_encap_to_wtap_encap(int encap)
 {
-       int i;
+       unsigned int i;
 
        for (i = 0; i < NUM_PCAP_ENCAPS; i++) {
                if (pcap_to_wtap_map[i].dlt_value == encap)
@@ -878,7 +878,7 @@ int wtap_pcap_encap_to_wtap_encap(int encap)
 
 static int wtap_wtap_encap_to_pcap_encap(int encap)
 {
-       int i;
+       unsigned int i;
 
        /*
         * Special-case WTAP_ENCAP_FDDI and WTAP_ENCAP_FDDI_BITSWAPPED;
index f0f266f31582a5ba33b2200de75dce94db093569..a0b489a71f53bce52c9124e48292f31a394c4465 100644 (file)
@@ -1,6 +1,6 @@
 /* netmon.c
  *
- * $Id: netmon.c,v 1.37 2001/03/10 06:33:57 guy Exp $
+ * $Id: netmon.c,v 1.38 2001/07/13 00:55:58 guy Exp $
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@xiexie.org>
@@ -126,7 +126,7 @@ int netmon_open(wtap *wth, int *err)
        int frame_table_size;
        guint32 *frame_table;
 #ifdef WORDS_BIGENDIAN
-       int i;
+       unsigned int i;
 #endif
 
        /* Read in the string that should be at the start of a Network
index 63475d73b065812783aa2dcaaeb50b8ff41fb6a4..db9bbae29312d11534185c979332abdfa2653f30 100644 (file)
@@ -1,6 +1,6 @@
 /* wtap.c
  *
- * $Id: wtap.c,v 1.53 2001/07/05 00:34:42 guy Exp $
+ * $Id: wtap.c,v 1.54 2001/07/13 00:55:58 guy Exp $
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@xiexie.org>
@@ -55,7 +55,7 @@ wtap_file_encap(wtap *wth)
 }
 
 /* Table of the encapsulation types we know about. */
-const static struct encap_type_info {
+static const struct encap_type_info {
        const char *name;
        const char *short_name;
 } encap_table[WTAP_NUM_ENCAP_TYPES] = {
@@ -183,7 +183,7 @@ const char
 *wtap_strerror(int err)
 {
        static char errbuf[128];
-       int wtap_errlist_index;
+       unsigned int wtap_errlist_index;
 
        if (err < 0) {
 #ifdef HAVE_LIBZ