diff options
author | Martin Mathieson <martin.r.mathieson@googlemail.com> | 2018-01-29 19:44:07 +0000 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2018-01-30 00:42:10 +0000 |
commit | 42322ac8cf2d99cb15d60a8d6d4053b19f6b60a5 (patch) | |
tree | e4c18910ea01f2bc4be2329b2b1cdb4aec840c60 | |
parent | 8342e5f97b7564ec423507453f7b5e4e5f48fffe (diff) |
dct2000 wiretap: expect all lines to have exactly 4 subsecond digits.
Should fix clang warning created by https://code.wireshark.org/review/#/c/25492.
Change-Id: Iafa31e24cd786a510f3a953d615df4cbc3930fa6
Reviewed-on: https://code.wireshark.org/review/25508
Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Jakub Zawadzki <darkjames-ws@darkjames.pl>
Reviewed-by: Michael Mann <mmann78@netscape.net>
-rw-r--r-- | epan/dissectors/packet-catapult-dct2000.c | 2 | ||||
-rw-r--r-- | wiretap/catapult_dct2000.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-catapult-dct2000.c b/epan/dissectors/packet-catapult-dct2000.c index d27a515ded..c26b098cdd 100644 --- a/epan/dissectors/packet-catapult-dct2000.c +++ b/epan/dissectors/packet-catapult-dct2000.c @@ -2193,7 +2193,7 @@ dissect_catapult_dct2000(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, vo multiplier *= 10; } - /* Subseconds (4 digits). */ + /* Subseconds (4 digits). N.B. trailing zeros are written out by wiretap module. */ int subseconds = 0; subseconds += (timestamp_string[timestamp_length-2]-'0'); subseconds += (timestamp_string[timestamp_length-3]-'0')*10; diff --git a/wiretap/catapult_dct2000.c b/wiretap/catapult_dct2000.c index c3e7dc63ac..3e609daf25 100644 --- a/wiretap/catapult_dct2000.c +++ b/wiretap/catapult_dct2000.c @@ -1214,8 +1214,8 @@ parse_line(gchar *linebuff, gint line_length, } subsecond_decimals_buff[subsecond_decimals_chars] = linebuff[n]; } - if (subsecond_decimals_chars > MAX_SUBSECOND_DECIMALS || n >= line_length) { - /* More numbers than expected - give up */ + if (subsecond_decimals_chars != MAX_SUBSECOND_DECIMALS || n >= line_length) { + /* There should be exactly 4 subsecond digits - give up if not */ return FALSE; } /* Convert found value into microseconds */ |