diff options
author | Guy Harris <gharris@sonic.net> | 2020-09-10 01:39:53 -0700 |
---|---|---|
committer | AndersBroman <a.broman58@gmail.com> | 2020-09-10 11:02:22 +0000 |
commit | b446e3647174fab41e5623b16d0ca31ca4898c90 (patch) | |
tree | 805ad188824587e972be474795f4183142b8663b | |
parent | 1b5be9bcba5986b5f4974fa0f3fcedf316a65afe (diff) |
aeron: the Error String in an Error Header is not null-terminated.
The Aeron specification says nothing about it being null-terminated, and
in at least some captures, it's not null terminated.
Make it an FT_STRING, rather than an FT_STRINGZ.
Clean up a comment so that more of the URL is visible in a narrower
window.
-rw-r--r-- | epan/dissectors/packet-aeron.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/epan/dissectors/packet-aeron.c b/epan/dissectors/packet-aeron.c index 0b1c721302..71cfd1eb24 100644 --- a/epan/dissectors/packet-aeron.c +++ b/epan/dissectors/packet-aeron.c @@ -19,7 +19,11 @@ #include <epan/to_str.h> #include <wsutil/pint.h> -/* The Aeron protocol is defined at https://github.com/real-logic/Aeron/wiki/Protocol-Specification */ +/* + * The Aeron protocol is defined at + * + * https://github.com/real-logic/aeron/wiki/Transport-Protocol-Specification + */ void proto_register_aeron(void); void proto_reg_handoff_aeron(void); @@ -3202,7 +3206,7 @@ void proto_register_aeron(void) { &hf_aeron_err_off_hdr, { "Offending Header", "aeron.err.off_hdr", FT_BYTES, BASE_NONE, NULL, 0x0, NULL, HFILL } }, { &hf_aeron_err_string, - { "Error String", "aeron.err.string", FT_STRINGZ, BASE_NONE, NULL, 0x0, NULL, HFILL } }, + { "Error String", "aeron.err.string", FT_STRING, BASE_NONE, NULL, 0x0, NULL, HFILL } }, { &hf_aeron_heartbeat, { "Heart Frame", "aeron.heartbeat", FT_NONE, BASE_NONE, NULL, 0x0, NULL, HFILL } }, { &hf_aeron_heartbeat_frame_length, |