diff options
| author | Martin Mathieson <martin.mathieson@keysight.com> | 2020-02-22 22:58:25 +0000 |
|---|---|---|
| committer | Martin Mathieson <martin.r.mathieson@googlemail.com> | 2020-02-23 21:43:51 +0000 |
| commit | ce0d780c3fc46501a9ee0c503adc41b5cc9d55ad (patch) | |
| tree | 584d0ab034cf468d9a6a1609b6f6c8a2f08431f8 /wiretap | |
| parent | a9d7ce53b191db62f4b9ed21b5c1a3ea1c102b96 (diff) | |
Some more issues spotted by PVS-Studio in bug 16335.
Trivial, mostly just redundant assignments or
format specifiers.
Change-Id: Iaf33f24d2af5a48a5e1b797e582bf936914c8daa
Reviewed-on: https://code.wireshark.org/review/36154
Petri-Dish: Martin Mathieson <martin.r.mathieson@googlemail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Martin Mathieson <martin.r.mathieson@googlemail.com>
Diffstat (limited to 'wiretap')
| -rw-r--r-- | wiretap/catapult_dct2000.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/wiretap/catapult_dct2000.c b/wiretap/catapult_dct2000.c index ddfdaaffe0..1f339e24e5 100644 --- a/wiretap/catapult_dct2000.c +++ b/wiretap/catapult_dct2000.c @@ -985,9 +985,9 @@ parse_line(gchar *linebuff, gint line_length, *encap = WTAP_ENCAP_RAW_IP; } - else /* FP may be carried over ATM, which has separate atm header to parse */ + else if ((strcmp(protocol_name, "fp") == 0) || (strncmp(protocol_name, "fp_r", 4) == 0)) { |