URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e6b1354d06e7ddf8f048109ee06d3a1e21941e8d Submitter: "Guy Harris <ghar...@sonic.net>" Changed: branch: master-3.0 Repository: wireshark
Commits: e6b1354 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> (backported from commit 7e02165453f78dc2196d66d6167d8fdeebdf2cb3) Reviewed-on: https://code.wireshark.org/review/37053 Petri-Dish: Guy Harris <ghar...@sonic.net> Reviewed-by: Guy Harris <ghar...@sonic.net> Actions performed: from 78ea376 Some issues spotted by PVS-Studio in bug 16335. Many more remain add e6b1354 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-sdp.c | 2 -- epan/dissectors/packet-sip.c | 1 - epan/dissectors/snort-config.c | 2 +- wiretap/catapult_dct2000.c | 1 - 6 files changed, 9 insertions(+), 20 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