Ethereal-cvs: [Ethereal-cvs] cvs commit: ethereal packet-aarp.c packet-arp.c packet-arp.h pack

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Guy Harris <guy@xxxxxxxxxxxxxxxxxxx>
Date: Mon, 13 Nov 2000 01:19:22 -0600 (CST)
guy         2000/11/13 01:19:20 CST

  Modified files:
    .                    packet-aarp.c packet-arp.c packet-arp.h 
                         packet-atm.c packet-cdp.c packet-ipp.c 
                         packet-kerberos.c packet-nfs.c 
                         packet-q2931.c packet-q931.c 
                         packet-snmp.c 
  Log:
  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.
  
  Revision  Changes    Path
  1.24      +2 -1      ethereal/packet-aarp.c
  1.34      +179 -170  ethereal/packet-arp.c
  1.4       +2 -2      ethereal/packet-arp.h
  1.25      +4 -4      ethereal/packet-atm.c
  1.26      +2 -1      ethereal/packet-cdp.c
  1.14      +2 -1      ethereal/packet-ipp.c
  1.5       +3 -1      ethereal/packet-kerberos.c
  1.38      +4 -4      ethereal/packet-nfs.c
  1.12      +10 -10    ethereal/packet-q2931.c
  1.20      +12 -11    ethereal/packet-q931.c
  1.52      +2 -1      ethereal/packet-snmp.c