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-ipp.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-ipp.c')
-rw-r--r-- | packet-ipp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/packet-ipp.c b/packet-ipp.c index a265b439b2..378d2de541 100644 --- a/packet-ipp.c +++ b/packet-ipp.c @@ -3,7 +3,7 @@ * * Guy Harris <guy@alum.mit.edu> * - * $Id: packet-ipp.c,v 1.13 2000/11/09 10:56:32 guy Exp $ + * $Id: packet-ipp.c,v 1.14 2000/11/13 07:18:46 guy Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -40,6 +40,7 @@ #include <glib.h> #include "packet.h" +#include "strutil.h" #include "packet-http.h" static int proto_ipp = -1; |