Wireshark-commits: [Wireshark-commits] master-3.2 0ed3a4b: Clean up a number of things.

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 03 Apr 2020 00:21:57 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0ed3a4bd252475250f4ab582718ff019d2b57635
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

0ed3a4b by Guy Harris (guy@xxxxxxxxxxxx):

    Clean up a number of things.
    
    Pass to the routines that handle particular ISIS PDU types a tvbuff for
    the *full* PDU; some PDU types may have a checksum CLV type that
    checksums the *entire* PDU.
    
    Pass an isis_data_t * around to various routines, rather than passing
    some individual bits of information around.
    
    Add to that structure:
    
    	the PDU length from the common-to-all-PDU-types part of the ISIS
    	header;
    
    	a proto_item * for the header length field;
    
    	an expert_field * for a "the header length is bad" error.
    
    Use the PDU length from that structure when handling the aforementioned
    checksum CLV.
    
    When dissecting the PDU-type-specific part of the ISIS header, check to
    make sure we're not going past the header length and, if we are, report
    it with an expert info, using the header length field proto_item * and
    expert_field * from that structure.
    
    Show the type field in sub-TLVs of the Group Address TLV (RFC 7176
    section 2.1) and, if the type is unknown, add a top-level item with the
    type and length fields under it.
    
    This fixes some bugs.
    
    Bug: 16477
    Change-Id: I875306d9d4fd8f65a60b7a6d3be7e356afabe851
    Reviewed-on: https://code.wireshark.org/review/36671
    Petri-Dish: Guy Harris <gharris@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    (cherry picked from commit 7af078d8fe0cddc874cc39d533f4f9fe7ac3b6b9)
    Reviewed-on: https://code.wireshark.org/review/36672
    

Actions performed:

    from  39eee85   Document the -k option.
     add  0ed3a4b   Clean up a number of things.


Summary of changes:
 epan/dissectors/packet-isis-clv.c   |  16 +-
 epan/dissectors/packet-isis-clv.h   |   6 +-
 epan/dissectors/packet-isis-hello.c | 141 +++++++++++------
 epan/dissectors/packet-isis-lsp.c   | 302 ++++++++++++++++++++++--------------
 epan/dissectors/packet-isis-snp.c   | 157 ++++++++++++-------
 epan/dissectors/packet-isis.c       |  21 ++-
 epan/dissectors/packet-isis.h       |   3 +
 7 files changed, 408 insertions(+), 238 deletions(-)