Wireshark-commits: [Wireshark-commits] master e30f0d0: Some 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: Sat, 15 Feb 2020 11:06:48 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=e30f0d09208b01a7dd3d685b612110d16683f045
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

e30f0d0 by Martin Mathieson (martin.mathieson@xxxxxxxxxxxx):

    Some issues spotted by PVS-Studio in bug 16335.  Many more remain
    
    Change-Id: If856e25af8e33eeef5b9e595f1f6820459892b17
    Reviewed-on: https://code.wireshark.org/review/36110
    Petri-Dish: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  36e5f95   UDPCP: Check signalled length against available payload
     add  e30f0d0   Some issues spotted by PVS-Studio in bug 16335.  Many more remain


Summary of changes:
 epan/dissectors/packet-catapult-dct2000.c |  1 -
 epan/dissectors/packet-kafka.c            |  6 ------
 epan/dissectors/packet-mac-lte.c          | 12 ++++++------
 epan/dissectors/packet-pdcp-lte.c         |  2 +-
 epan/dissectors/packet-rlc-lte.c          |  2 +-
 epan/dissectors/packet-snort.c            |  4 ++--
 epan/dissectors/packet-tftp.c             |  2 +-
 epan/dissectors/packet-umts_fp.c          | 12 +++++-------
 ui/tap-rlc-graph.c                        |  3 ++-
 9 files changed, 18 insertions(+), 26 deletions(-)