Ethereal-dev: Re: [Ethereal-dev] NULL for the blurb for a header field?

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

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Gerald Combs <gerald@xxxxxxxxxxxx>
Date: Fri, 11 Nov 2005 14:25:00 -0600
Stig Bj�rlykke wrote:
> On 11. nov. 2005, at 05.34, Guy Harris wrote:
> 
>> Currently, a lot of dissectors have null strings for blurbs; should we
>> allow a null pointer for the blurb, meaning "sorry, nobody's written a
>> blurb for this field" (which is what a null string means), and add 
>> checks
>> for a null blurb pointer?
> 
> 
> Have a look in the developer's guide, chapter 8.4 "How to reassemble 
> split packets" (Example 8.21. Reassembling fragments - Data), where 
> this code came from.  I think code examples from this guide should be 
> valid according to current rules :)

We need to fix one or more of: the Developer's Guide, Ethereal's
blurb-handling code, or Solaris' inability to handle null string
pointers in printf().