URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e6b1354d06e7ddf8f048109ee06d3a1e21941e8d
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark
Commits:
e6b1354 by Martin Mathieson (martin.mathieson@xxxxxxxxxxxx):
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.mathieson@xxxxxxxxxxxxxx>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
(backported from commit 7e02165453f78dc2196d66d6167d8fdeebdf2cb3)
Reviewed-on: https://code.wireshark.org/review/37053
Petri-Dish: Guy Harris <gharris@xxxxxxxxx>
Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
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(-)