Home page logo
/

wireshark logo Wireshark mailing list archives

Re: [Wireshark-commits] rev 49705: /trunk/wiretap/ /trunk/wiretap/: peektagged.c
From: Guy Harris <guy () alum mit edu>
Date: Sun, 2 Jun 2013 19:19:38 -0700


On Jun 2, 2013, at 6:07 PM, eapache () wireshark org wrote:

Initialize some variables that GCC 4.7 complains about. I'm not 100% sure
that the complaints are valid,

For hdr_info.length, they're not - we error out if there's no length field.

For hdr_info.sliceLength, they are.

or that simply zeroing them is the right fix if they are,

It's the best workaround for compilers whose dataflow analysis isn't sophisticated enough.

but at least it builds now. Should we be erroring if we don't
see a sliceLength header?

I'm not sure - currently, now that hdr_info.sliceLength is zeroed before we look at the attribute/value pairs, and 
given that, if it's zero, we assume no slicing, the code works.

We don't have a published spec for the file format, so we don't know whether the slice length attribute is optional or 
not.

(In the hopes of preempting replies saying "WildPackets does publish the file format, see their PeekRdr application", 
note that this page:

        https://mypeek.wildpackets.com/view_solution.php?id=26

says "You must have a valid maintenance contract to download this file."  Unless there's some way to download a 
description of the file format *without* purchasing OmniPeek or a maintenance contract for it, or a way for somebody 
who has downloaded it to *legally* provide us with it - e.g., if you signed some form of non-disclosure agreement when 
you purchased OmniPeek or the maintenance contract and that NDA covers that document, or if the document says "you 
can't give this away" or "you can only give this away to people who have a maintenance contract for OmniPeek", that's 
of no use to us.)
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe


  By Date           By Thread  

Current thread:
  • Re: [Wireshark-commits] rev 49705: /trunk/wiretap/ /trunk/wiretap/: peektagged.c Guy Harris (Jun 03)
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]
AlienVault