Wireshark-dev: Re: [Wireshark-dev] FW: DISSECTOR_ASSERT_NOT_REACHED in WLCCP decode...

From: "Kevin A. Noll" <kevin.noll@xxxxxxxxxxxxx>
Date: Wed, 30 May 2007 16:40:31 -0400
I think I understand... It's sort of like a null-terminated string, but it's
a null-terminated array?

--kan-- 



--
Kevin A. Noll, KD4WOZ
CCIE, CCDP
Versatile, Inc.		
Kevin.Noll@xxxxxxxxxxxxx
+1-717-796-1936

-----Original Message-----
From: wireshark-dev-bounces@xxxxxxxxxxxxx
[mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Joerg Mayer
Sent: Wednesday, May 30, 2007 4:18 PM
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] FW: DISSECTOR_ASSERT_NOT_REACHED in WLCCP
decode...

On Wed, May 30, 2007 at 01:15:35PM -0700, Stephen Fisher wrote:
> > So I'm looking at the value strings, and I'm wondering why we should 
> > terminate them with {0, NULL} and what happens if one of the value 
> > pairs needs to be {0, "a real string"} ?
> 
> You can still use 0, "a real string" as one of the entries.  You just 
> need to have 0, NULL as the final entry.  If you don't, the code will 
> keep reading past the end and run into random memory space looking for 
> that 0, NULL entry.

And one of those overruns might actually cause the crash you were talking
about.

 Cia
 o  Joerg

-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev