diff options
author | Guy Harris <guy@alum.mit.edu> | 2018-01-11 01:31:08 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2018-01-11 09:31:43 +0000 |
commit | eb5d12c4b97b11c3dcf16cc9d245b631f4389e59 (patch) | |
tree | 3380996d66a15f2fdf11551a5fc864b3f012db53 | |
parent | 3d38b87d86736a976dd28f03b989282e093191bb (diff) |
For AT_STRINGZ, the length *includes* the null terminator.
The correct length for an AT_STRINGZ address of "" is 1, not 0. A
length of 0 for an address is valid only if the pointer-to-address-data
is null.
Change-Id: I1da6de5ed402020ed5c8389a911870a54fa8b14a
Reviewed-on: https://code.wireshark.org/review/25258
Reviewed-by: Guy Harris <guy@alum.mit.edu>
-rw-r--r-- | epan/dissectors/packet-logcat-text.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-logcat-text.c b/epan/dissectors/packet-logcat-text.c index e13614499a..9d01b8ba62 100644 --- a/epan/dissectors/packet-logcat-text.c +++ b/epan/dissectors/packet-logcat-text.c @@ -257,8 +257,8 @@ static int dissect_logcat_text_process(tvbuff_t *tvb, packet_info *pinfo, proto_ dissect_info_t dinfo = { &process_regex, getters, array_length(getters) }; add_exported_pdu(tvb,pinfo,"logcat_text_process"); - set_address(&pinfo->dst, AT_STRINGZ, 0, ""); - set_address(&pinfo->src, AT_STRINGZ, 0, ""); + set_address(&pinfo->dst, AT_STRINGZ, 1, ""); + set_address(&pinfo->src, AT_STRINGZ, 1, ""); return dissect_logcat_text(tvb, tree, pinfo, &dinfo); } |