Wireshark-commits: [Wireshark-commits] master ab53831: Netmon+Message Analyzer: Fix a couple of len
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: Mon, 24 Sep 2018 20:08:50 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ab53831abd962d4fc57e58fe3a7cc18c523c543c
Submitter: "Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

ab53831 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Netmon+Message Analyzer: Fix a couple of lengths.
    
    Set the length of the Network Monitor Event and ETW Ndis protocols to
    that of their respective headers instead of the remaining TVB length.
    
    Add packet-netmon.c and packet-messageanalyzer.c to .editorconfig.
    
    Change-Id: I50c23c63a74190cebf760fa6b374cc1b78c0191e
    Reviewed-on: https://code.wireshark.org/review/29810
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  1c0d066   sharkd: cleanup printf() usage during JSON generation (part II).
     add  ab53831   Netmon+Message Analyzer: Fix a couple of lengths.


Summary of changes:
 epan/dissectors/.editorconfig            | 8 ++++++++
 epan/dissectors/packet-messageanalyzer.c | 4 ++--
 epan/dissectors/packet-netmon.c          | 2 ++
 3 files changed, 12 insertions(+), 2 deletions(-)