diff options
| author | Guy Harris <guy@alum.mit.edu> | 2000-11-13 07:19:37 +0000 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2000-11-13 07:19:37 +0000 |
| commit | 99c98f9e74d8d812e0e87c960970116bf149434b (patch) | |
| tree | d3b2949261e9970321c2a8fac4be3d2baaeb63c1 /packet-aarp.c | |
| parent | 796997a538eede8129af9fbb938b3b1cc8bc719d (diff) | |
| download | wireshark-99c98f9e74d8d812e0e87c960970116bf149434b.tar.gz wireshark-99c98f9e74d8d812e0e87c960970116bf149434b.tar.bz2 wireshark-99c98f9e74d8d812e0e87c960970116bf149434b.zip | |
Move "bytes_to_str()" to "strutil.c" from "packet.c" - it's just a
string formatter, like "format_text()", and, as "tvbuff.c" now calls it
(*vide infra*), we don't want to have to make "tvbuff.c" drag "packet.h"
in just to declare "bytes_to_str()". It's now declared in "strutil.h",
so include it in modules that use "bytes_to_str()" and weren't already
including it.
Add a "tvb_bytes_to_str()" wrapper that calls "tvb_get_ptr()" to get a
pointer to a chunk of N bytes at a given offset in a tvbuff and then
hands that chunk to "bytes_to_str()". Convert the code that was doing
that to use "tvb_bytes_to_str()" instead (which caught what I suspect is
a bug in the Q.2931 dissector, where it was handing an offset of 0 to
"tvb_get_ptr()" - a cut-and-pasteo, I think).
Tvbuffify the ARP dissector.
svn path=/trunk/; revision=2634
Diffstat (limited to 'packet-aarp.c')
| -rw-r--r-- | packet-aarp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/packet-aarp.c b/packet-aarp.c index fe56cc5144..d32ba74696 100644 --- a/packet-aarp.c +++ b/packet-aarp.c @@ -1,7 +1,7 @@ /* packet-aarp.c * Routines for Appletalk ARP packet disassembly * - * $Id: packet-aarp.c,v 1.23 2000/11/13 04:26:53 guy Exp $ + * $Id: packet-aarp.c,v 1.24 2000/11/13 07:18:37 guy Exp $ * * Simon Wilkinson <sxw@dcs.ed.ac.uk> * @@ -31,6 +31,7 @@ #include <stdio.h> #include <glib.h> #include "packet.h" +#include "strutil.h" #include "etypes.h" static int proto_aarp = -1; |