Wireshark-commits: [Wireshark-commits] master-3.2 91a621e: More issues spotted by PVS-Studio in bug
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, 03 May 2020 05:27:43 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=91a621e721a37276f2403f6b25bc81680cec4265
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

91a621e 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>
    (cherry picked from commit 7e02165453f78dc2196d66d6167d8fdeebdf2cb3)
    Reviewed-on: https://code.wireshark.org/review/37052
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    

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(-)