Wireshark-commits: [Wireshark-commits] master-3.2 92d657d: Some 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 01:45:00 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=92d657d98b6e466d8212a862191744bb9d8f80de
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

92d657d 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>
    (backported from commit e30f0d09208b01a7dd3d685b612110d16683f045)
    Reviewed-on: https://code.wireshark.org/review/37049
    Petri-Dish: Guy Harris <gharris@xxxxxxxxx>
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    

Actions performed:

    from  c086ce9   prefs: plug a memory leak.
     add  92d657d   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-umts_fp.c          | 12 +++++-------
 ui/tap-rlc-graph.c                        |  3 ++-
 8 files changed, 17 insertions(+), 25 deletions(-)