Wireshark mailing list archives
Re: [PATCH] add support for tipc dissector at transport level.
From: Jeff Morriss <jeff.morriss.ws () gmail com>
Date: Fri, 27 Jul 2012 10:29:53 -0400
IANA is definitely the better way to go, but another option is to make the port number configurable (as a dissector preference, maybe using a "range" preference in case multiple ports are needed), possibly defaulting to 0 (meaning users must configure it to get it to work).
Making the dissector a "new style" one that does heuristics and does not accept packets that don't appear to be its protocol would make this even better.
Ideally this should be done via a bug report so that sample captures of such TIPC-over-UDP traffic could be attached for automatic fuzz testing.
Erik Hugne wrote:
Good question. I'll bring this up with the TIPC working group. //E On 2012-07-27 14:01, Jaap Keuter wrote:Hi, Why not request for a proper portnumber at IANA? Thanks, Jaap On 2012-07-27 13:32, Karin Nilsson wrote:Signed-off-by: Karin Nilsson <karin.nilsson () ericsson com> --- epan/dissectors/packet-tipc.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/epan/dissectors/packet-tipc.c b/epan/dissectors/packet-tipc.c index 949cdf3..864d789 100644 --- a/epan/dissectors/packet-tipc.c +++ b/epan/dissectors/packet-tipc.c @@ -28,7 +28,7 @@ * Protocol ref: * http://tipc.sourceforge.net/ */ - +#define TIPC_UDP_DEFAULT_PORT 54711 #ifdef HAVE_CONFIG_H # include "config.h" @@ -2954,6 +2954,7 @@ proto_reg_handoff_tipc(void) data_handle = find_dissector("data"); dissector_add_uint("ethertype", ETHERTYPE_TIPC, tipc_handle); + dissector_add_uint("udp.port", TIPC_UDP_DEFAULT_PORT, tipc_handle); inited = TRUE; } else {___________________________________________________________________________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-devmailto:wireshark-dev-request () wireshark org?subject=unsubscribe___________________________________________________________________________ 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-request () wireshark org?subject=unsubscribe
___________________________________________________________________________ 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-request () wireshark org?subject=unsubscribe
Current thread:
- [PATCH] add support for tipc dissector at transport level. Karin Nilsson (Jul 27)
- Re: [PATCH] add support for tipc dissector at transport level. Jaap Keuter (Jul 27)
- Re: [PATCH] add support for tipc dissector at transport level. Erik Hugne (Jul 27)
- Re: [PATCH] add support for tipc dissector at transport level. Jeff Morriss (Jul 27)
- Re: [PATCH] add support for tipc dissector at transport level. Erik Hugne (Jul 30)
- Re: [PATCH] add support for tipc dissector at transport level. Jaap Keuter (Jul 31)
- Re: [PATCH] add support for tipc dissector at transport level. Erik Hugne (Jul 27)
- Re: [PATCH] add support for tipc dissector at transport level. Jaap Keuter (Jul 27)