diff options
| author | Michael Mann <mmann78@netscape.net> | 2016-10-05 16:33:54 -0400 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2016-10-13 02:51:18 +0000 |
| commit | 2eb7b05b8c9c6408268f0d1e81f0a18a02610f1c (patch) | |
| tree | bd0a909e3d506ea2e61c446aa8a809b6f728af3b /plugins/profinet | |
| parent | 4d8581d7e15fe4a80a53496b83a853271fc674b6 (diff) | |
Convert most UDP dissectors to use "auto" preferences.
Similar to the "tcp.port" changes in I99604f95d426ad345f4b494598d94178b886eb67,
convert dissectors that use "udp.port".
More cleanup done on dissectors that use both TCP and UDP dissector
tables, so that less preference callbacks exist.
Change-Id: If07be9b9e850c244336a7069599cd554ce312dd3
Reviewed-on: https://code.wireshark.org/review/18120
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'plugins/profinet')
| -rw-r--r-- | plugins/profinet/packet-pn-rt.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/plugins/profinet/packet-pn-rt.c b/plugins/profinet/packet-pn-rt.c index 885cd9fc68..6ee55eb67c 100644 --- a/plugins/profinet/packet-pn-rt.c +++ b/plugins/profinet/packet-pn-rt.c @@ -39,6 +39,8 @@ void proto_register_pn_rt(void); void proto_reg_handoff_pn_rt(void); +#define PROFINET_UDP_PORT 0x8892 + /* Define the pn-rt proto */ static int proto_pn_rt = -1; static gboolean pnio_desegment = TRUE; @@ -1001,7 +1003,7 @@ proto_reg_handoff_pn_rt(void) pn_rt_handle = create_dissector_handle(dissect_pn_rt, proto_pn_rt); dissector_add_uint("ethertype", ETHERTYPE_PROFINET, pn_rt_handle); - dissector_add_uint("udp.port", 0x8892, pn_rt_handle); + dissector_add_uint_with_preference("udp.port", PROFINET_UDP_PORT, pn_rt_handle); heur_dissector_add("pn_rt", dissect_CSF_SDU_heur, "PROFINET CSF_SDU IO", "pn_csf_sdu_pn_rt", proto_pn_rt, HEURISTIC_ENABLE); heur_dissector_add("pn_rt", dissect_FRAG_PDU_heur, "PROFINET Frag PDU IO", "pn_frag_pn_rt", proto_pn_rt, HEURISTIC_ENABLE); |