Wireshark-dev: Re: [Wireshark-dev] [Wireshark-bugs] [Bug 6485] Buildbot crash output: fuzz-2011

From: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>
Date: Fri, 04 Nov 2011 11:24:43 -0400
bugzilla-daemon@xxxxxxxxxxxxx wrote:
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=6485

--- Comment #8 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2011-10-27 11:12:52 PDT ---
Oh good grief.  Ignore the backtrace in comment 7, too. That's caused by me
using an out-of-tree build and bug 5664 (so my tshark wasn't picking up the
radius files which eventually causes a DISSECTOR_ASSERT).

The REAL problem is that the GSM_MAP dissector is using this value_string_ext
in the hf without BASE_EXT_STRING:

    { &hf_gsm_old_localValue,
      { "localValue", "gsm_old.localValue",
        FT_INT32, BASE_DEC, &gsm_old_GSMMAPOperationLocalvalue_vals_ext, 0,
        "OperationLocalvalue", HFILL }},

This, in turn, appears to be caused because OperationLocalValue is an alias
for/of GSMMAPOperationLocalValue and only the latter is defined with
.USE_VALS_EXT.

I can fix it by doing:

Index: asn1/gsm_map/gsm_map.cnf
===================================================================
--- asn1/gsm_map/gsm_map.cnf    (revision 39628)
+++ asn1/gsm_map/gsm_map.cnf    (working copy)
@@ -54,6 +54,7 @@

 #.USE_VALS_EXT
 GSMMAPOperationLocalvalue
+OperationLocalvalue

 #.EXPORTS
 AddressString


But it seems to be that asn2wrs should arguably be figuring this out on its
own.

Can someone knowledgeable in asn2wrs take a look?

Just in case this got lost in the pile of fuzz failures recently... Can someone knowledgeable in asn2wrs take a look at this bug?