Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 52905: /trunk/epan/ /trunk/epan/: pr
From: Evan Huus <[email protected]>
Date: Sun, 27 Oct 2013 18:30:22 -0400
On Sun, Oct 27, 2013 at 5:56 PM,  <[email protected]> wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=52905
>
> User: morriss
> Date: 2013/10/27 09:56 PM
>
> Log:
>  Fix https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=9323 :
>
>  Actually handle non-NULL-terminated FT_STRINGs properly.
>
> Directory: /trunk/epan/
>   Changes    Path          Action
>   +8 -8      proto.c       Modified
>
> Directory: /trunk/epan/ftypes/
>   Changes    Path              Action
>   +5 -3      ftype-string.c    Modified

Thanks for your help tracking this one down. I think this accidentally
introduces a leak for NULL strings  though, because

fvalue_set(&fi->value, (gpointer) g_strdup("[ Null ]"), FALSE);

causes string_fvalue_set to take a second duplicate, leaking the
first. Either set already_copied to TRUE or remove the g_strdup here.

Evan