Wireshark-commits: [Wireshark-commits] master-3.0 e6b1354: More issues spotted by PVS-Studio in bug

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 03 May 2020 06:08:29 +0000
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(-)