Wireshark-commits: [Wireshark-commits] master 7e02165: More issues spotted by PVS-Studio in bug 163
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 16 Feb 2020 11:52:02 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=7e02165453f78dc2196d66d6167d8fdeebdf2cb3
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

7e02165 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>
    

Actions performed:

    from  fbd8849   [Automatic update for 2020-02-16]
     add  7e02165   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(-)