URL:
https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=91a621e721a37276f2403f6b25bc81680cec4265
Submitter: "Guy Harris <ghar...@sonic.net>"
Changed: branch: master-3.2
Repository: wireshark
Commits:
91a621e by Martin Mathieson (martin.mathie...@keysight.com):
More issues spotted by PVS-Studio in bug 16335.
Only the change to packet-imap.c really represents a bug.
Change-Id: Ie270f97f3d94c338ea3c84a712f8f4d43ffd36f4
Reviewed-on: https://code.wireshark.org/review/36115
Petri-Dish: Martin Mathieson <martin.r.mathie...@googlemail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broma...@gmail.com>
(cherry picked from commit 7e02165453f78dc2196d66d6167d8fdeebdf2cb3)
Reviewed-on: https://code.wireshark.org/review/37052
Reviewed-by: Guy Harris <ghar...@sonic.net>
Actions performed:
from 92d657d Some issues spotted by PVS-Studio in bug 16335. Many more
remain
add 91a621e More issues spotted by PVS-Studio in bug 16335.
Summary of changes:
epan/dissectors/packet-ber.c | 15 ++++++---------
epan/dissectors/packet-ftp.c | 8 ++------
epan/dissectors/packet-imap.c | 2 +-
epan/dissectors/packet-sdp.c | 2 --
epan/dissectors/packet-sip.c | 1 -
epan/dissectors/packet-snort-config.c | 2 +-
wiretap/catapult_dct2000.c | 1 -
7 files changed, 10 insertions(+), 21 deletions(-)
___________________________________________________________________________
Sent via: Wireshark-commits mailing list <wireshark-commits@wireshark.org>
Archives: https://www.wireshark.org/lists/wireshark-commits
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-commits
mailto:wireshark-commits-requ...@wireshark.org?subject=unsubscribe
