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-boun...@wireshark.org 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 <wme...@newsguy.com> 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@wireshark.org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?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.
<<winmail.dat>>
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe