ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-bugs: [Wireshark-bugs] [Bug 2974] packet-radius.c - avp specific dissectors broken

Date: Fri, 17 Oct 2008 08:08:13 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2974





--- Comment #3 from Bill Meier <wmeier@xxxxxxxxxxx>  2008-10-17 08:08:12 PDT ---
After looking at this:

I think the problem is that register_radius_fields() (which does the dictionary
load) is being registered as a "delayed" field registration routine (via
proto_register_prefix) and is also called during an actual dissection.
Therefore it would see that register_radius_fields [and thus
radius_load_dictionary()]is called *after* all the proto_reg_handoff functions
and thus apparently messes up any and all radius_register_avp_dissector calls
done in proto_reg_handoff...

Thus: I suspect the patch fixes things for packet_radius itself but not for any
other dissector which calls radius_register_avp_dissector() in a
proto_reg_handoff function.

I'll leave this to someone who knows what packet-radius.c is doing.... (I've no
clue).

Bill


-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.