Wireshark-commits: [Wireshark-commits] master ce0d780: Some more issues spotted by PVS-Studio in bu
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, 23 Feb 2020 21:43:59 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ce0d780c3fc46501a9ee0c503adc41b5cc9d55ad
Submitter: "Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

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

    Some more issues spotted by PVS-Studio in bug 16335.
    
    Trivial, mostly just redundant assignments or
    format specifiers.
    
    Change-Id: Iaf33f24d2af5a48a5e1b797e582bf936914c8daa
    Reviewed-on: https://code.wireshark.org/review/36154
    Petri-Dish: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    

Actions performed:

    from  a9d7ce5   FTDI FT: Add Amontec JTAGkey USB VID/PID
     add  ce0d780   Some more issues spotted by PVS-Studio in bug 16335.


Summary of changes:
 epan/dissectors/packet-http2.c   |  2 --
 epan/dissectors/packet-mac-lte.c | 20 ++++++++++----------
 epan/dissectors/packet-netflow.c |  3 +--
 epan/dissectors/packet-nfapi.c   | 12 ++++++------
 ui/tap-rlc-graph.c               |  2 --
 ui/tap-tcp-stream.c              |  1 -
 wiretap/catapult_dct2000.c       |  2 +-
 7 files changed, 18 insertions(+), 24 deletions(-)