Wireshark-dev: Re: [Wireshark-dev] buildbot failure in Wireshark (development) on Solaris-10-SP

From: didier <dgautheron@xxxxxxxx>
Date: Thu, 20 Aug 2009 00:47:55 +0200
Hi,
Le mercredi 19 aoᅵt 2009 ᅵ 13:55 -0700, Guy Harris a ᅵcrit :
> On Aug 19, 2009, at 1:28 PM, buildbot-no-reply@xxxxxxxxxxxxx wrote:
> 
> > The Buildbot has detected a new failure of Solaris-10-SPARC on  
> > Wireshark (development).
> > Full details are available at:
> > http://buildbot.wireshark.org/trunk/builders/Solaris-10-SPARC/builds/2219
> >
> > Buildbot URL: http://buildbot.wireshark.org/trunk/
> >
> > Buildslave for this Build: solaris-10-sparc
> >
> > Build Reason:
> > Build Source Stamp: HEAD
> > Blamelist: etxrab
> >
> > BUILD FAILED: failed compile
> 
> g_string_chunk_clear() is documented as being "Since 2.14"; should we  
> find another way of freeing the strings (would freeing the entire  
> string chunk, allocating a new chunk, and setting the pointer in  
> question to point to the new chunk work),
Yest it should work.

But there's a least another problem in 
gtk/expert_dlg.c:expert_dlg_packet()
the useless line.

 ei->protocol = ei->protocol;

In my version it works because I also changed epan/expert.c
expert_set_info_vformat()
ei->protocol    = ep_strdup(pinfo->current_proto);
to 
ei->protocol    = pinfo->current_proto;

because pinfo->current_proto is currently always a constant but it's not
something we can enforce. Maybe it should use a string_chunk too.

Didier