diff options
| -rw-r--r-- | epan/dissectors/packet-prp.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/epan/dissectors/packet-prp.c b/epan/dissectors/packet-prp.c index c0781269a8..5cf0f9c3d5 100644 --- a/epan/dissectors/packet-prp.c +++ b/epan/dissectors/packet-prp.c @@ -97,6 +97,13 @@ dissect_prp_redundancy_control_trailer(tvbuff_t *tvb, packet_info *pinfo _U_, pr * This is horribly broken. It assumes the frame is an Ethernet * frame, with a type field at an offset of 12 bytes from the header. * That is not guaranteed to be true. + * + * Ideally, this should be a heuristic dissector registered in + * the "eth.trailer" heuristic dissector table (and it can + * be registered as "disabled by default" there); unfortunately, + * it needs to know the length of the entire frame for the + * PRP-0 heuristic, so it'd have to be passed that length + * out of band. */ if (!tvb_bytes_exist(tvb, 12, 2)) return 0; |