diff options
| author | guy <guy> | 2002-09-05 21:25:34 +0000 |
|---|---|---|
| committer | guy <guy> | 2002-09-05 21:25:34 +0000 |
| commit | c422d3ab0f6de2d38512a4566637bc47df291e74 (patch) | |
| tree | 2d4881bf0fd7ad4e433201643b5b6a66f17e997c /print-llc.c | |
| parent | c2bfaa85753ea1cad8c3729692a6bdbb80780f91 (diff) | |
| download | tcpdump-c422d3ab0f6de2d38512a4566637bc47df291e74.tar.gz | |
Get rid of the "-Wno-unused" flag, and fix up most of the
unused-parameter problems reported by GCC. Add an _U_ tag to label
parameters as unused if the function is called through a pointer (so
that you can't change its signature by removing parameters) or if there
are unused parameters only because the function isn't complete.
Add some additional bounds checks the necessity for which was revealed
while cleaning up unused-parameter problems.
Make some routines static.
"lcp_print()", defined in "print-lcp.c", isn't called anywhere -
"print-ppp.c" has the code to dissect LCP. Get rid of "print-lcp.c".
Diffstat (limited to 'print-llc.c')
| -rw-r--r-- | print-llc.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/print-llc.c b/print-llc.c index 73a0ee72..cbdd0390 100644 --- a/print-llc.c +++ b/print-llc.c @@ -24,7 +24,7 @@ #ifndef lint static const char rcsid[] = - "@(#) $Header: /tcpdump/master/tcpdump/print-llc.c,v 1.49 2002-09-05 00:00:15 guy Exp $"; + "@(#) $Header: /tcpdump/master/tcpdump/print-llc.c,v 1.50 2002-09-05 21:25:44 guy Exp $"; #endif #ifdef HAVE_CONFIG_H @@ -193,8 +193,8 @@ llc_print(const u_char *p, u_int length, u_int caplen, * Does anybody ever bridge one form of LAN traffic * over a networking type that uses 802.2 LLC? */ - ret = snap_print(p, length, caplen, esrc, edst, - extracted_ethertype, orgcode, et, 2); + ret = snap_print(p, length, caplen, extracted_ethertype, + orgcode, et, 2); if (ret) return (ret); } @@ -286,8 +286,8 @@ llc_print(const u_char *p, u_int length, u_int caplen, int snap_print(const u_char *p, u_int length, u_int caplen, - const u_char *esrc, const u_char *edst, u_short *extracted_ethertype, - u_int32_t orgcode, u_short et, u_int bridge_pad) + u_short *extracted_ethertype, u_int32_t orgcode, u_short et, + u_int bridge_pad) { register int ret; @@ -325,7 +325,7 @@ snap_print(const u_char *p, u_int length, u_int caplen, case OUI_CISCO: if (et == PID_CISCO_CDP) { - cdp_print(p, length, caplen, esrc, edst); + cdp_print(p, length, caplen); return (1); } break; |