Wireshark-dev: Re: [Wireshark-dev] Custom dissector causing wireshark to crash
From: Patrick Lannigan <p.lannigan@xxxxxxxxx>
Date: Wed, 26 Aug 2009 09:59:40 -0400
Ok, the base issues were the problem and Wireshark now open successfully. It said mac.priority because I didn't check carefully enough when I renamed the protocol.

Patrick

On Tue, Aug 25, 2009 at 9:41 PM, Maynard, Chris <Christopher.Maynard@xxxxxxxxx> wrote:
You are using BASE_NONE for integers.  This isn't allowed.  Ref: doc/README.developer.

BTW, any reason why one of your display filters is called "mac.priority" and not "abc.priority"?

- Chris

________________________________

From: wireshark-dev-bounces@xxxxxxxxxxxxx on behalf of Patrick Lannigan
Sent: Tue 8/25/2009 6:09 PM
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] Custom dissector causing wireshark to crash


I fixed the change you mentioned and am still failing at the same proto_register_field_array call.


       static hf_register_info hf[] = {
       { &hf_abc_flags,
           { "Flags", "abc.flags",
           FT_UINT8, BASE_NONE,
           NULL, 0x0, NULL, HFILL }
       },
       { &hf_abc_crc,
           { "CRC", "abc.crc",
           FT_UINT16, BASE_HEX,
           NULL, 0x0, NULL, HFILL }
       },
       { &hf_abc_txPower,
           { "txPower", "abc.txPower",
           FT_UINT8, BASE_DEC,
           NULL, 0x0,
           "Transmit power for message in dBm",
           HFILL }
       },
       { &hf_abc_frameType,
           { "frameType", "abc.frameType",
           FT_UINT16, BASE_NONE,
           NULL, 0x0, NULL, HFILL }
       },
       { &hf_abc_prevHop,
           { "prevHop", "abc.prevHop",
           FT_UINT32, BASE_HEX,
           NULL, 0x0,
           "Address of node that message was received from",
           HFILL }
       },
       { &hf_abc_nextHop,
           { "nextHop", "abc.nextHop",
           FT_UINT32, BASE_HEX,
           NULL, 0x0,
           "Address of node that message is being sent to",
           HFILL }
       },
       { &hf_abc_seqNum,
           { "seqNum", "abc.seqNum",
           FT_UINT8, BASE_DEC,
           NULL, 0x0, NULL, HFILL }
       },
       { &hf_abc_priority,
           { "priority", "mac.priority",
           FT_UINT8, BASE_NONE,
           NULL, 0x0, NULL, HFILL }
       },
       { &hf_abc_frameContentLen,
           { "frameContentLen", "abc.frameContentLen",
           FT_UINT16, BASE_DEC,
           NULL, 0x0,
           "Length of buffer sent OTA including mac headers",
           HFILL }
       },
       { &hf_abc_duration,
           { "duration", "abc.duration",
           FT_UINT16, BASE_DEC,
           NULL, 0x0,
           "Duration in microsec used for CSMA NAV calculation",
           HFILL }
       },
       { &hf_abc_token,
           { "token", "abc.token",
           FT_UINT32, BASE_NONE,
           NULL, 0x0, NULL, HFILL }
       },
       { &hf_abc_linkProfile,
           { "linkProfile", "abc.linkProfile",
           FT_UINT32, BASE_NONE,
           NULL, 0x0,
           "Linkprofile used to xmit this packet, tmp until we know how the PHY is going to provide this info",
           HFILL }
       },
       { &hf_abc_expirationTime,
           {"expirationTime", "abc.expirationTime",
           FT_UINT16, BASE_DEC,
           NULL, 0x0,
           "Time in ms this frame may take to be delivered to its next hop",
           HFILL }
       },
       { &hf_abc_padding,
           {"padding", "abc.padding",
           FT_UINT16, BASE_NONE,
           NULL, 0x0, NULL, HFILL }
       }
   };

Patrick


On Tue, Aug 25, 2009 at 5:28 PM, Bill Meier <wmeier@xxxxxxxxxxx> wrote:


       > It then fails on:
       >
       > proto_register_field_array (proto_abc, hf, array_length (hf));
       >
       > Here is the hf array construction, can anyone see what the error is?
       >
       >     static hf_register_info hf[] = {
        >


       >         { &hf_abc_txPower,
       >             { "txPower", "abc.txPower",
       >             FT_UINT8, BASE_DEC,
       >             "Transmit power for message in dBm",
       >             0x0, NULL, HFILL }
       >         },


       should be


                { &hf_abc_txPower,
                    { "txPower", "abc.txPower",

                    FT_UINT8, BASE_DEC, NULL, 0x0,
                    "Transmit power for message in dBm", HFILL }
                },

       Stepping through the code (eg: proto_register-field_array) is the way to
       find bugs like this.



       ___________________________________________________________________________
       Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
       Archives:    http://www.wireshark.org/lists/wireshark-dev
       Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
                   mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe



CONFIDENTIALITY NOTICE: The contents of this email are confidential
and for the exclusive use of the intended recipient. If you receive this
email in error, please delete it from your system immediately and
notify us either by email, telephone or fax. You should not copy,
forward, or otherwise disclose the content of the email.

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
            mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe