Wireshark-commits: [Wireshark-commits] master cebf038: CIP: Attribute updates
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 10 Oct 2016 17:27:53 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cebf038721bd795f754df955c8cbbbb509e75122
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

cebf038 by D. Ulis (daulis0@xxxxxxxxx):

    CIP: Attribute updates
    
    1. Combine various Attribute ID filters into a single one "cip.attribute", so that it's easier to find a certain attribute in any part of the packet. This makes sense because Attributes refer to the same property, but various types of messages can have that field in different locations. This applies to Get/Set Attribute Single, Get/Set Attribute List, Get Attributes All.
    2. Connection Manager: Dissect more attributes
    3. Port: Display the Port Type name
    4. Get/Set Attribute Single: Show unparsed data as raw bytes. Previously, if there were extra bytes they were not displayed.
    5. Get/Set Attribute List: Show remaining data for unknown attributes under the main Attribute List tree, instead of under the unknown Attribute ID tree.
    6. Display Attribute ID as DEC, since the specification tables are in decimal.
    7. Update Class list
    8. Minor: Remove some unused variables, typos
    
    Change-Id: I1a65da5f827484a3cd0e49ec944572ea5a166db1
    Reviewed-on: https://code.wireshark.org/review/18118
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  e1059891  fix: use ws_strtoi function.
    adds  cebf038   CIP: Attribute updates


Summary of changes:
 epan/dissectors/packet-cip.c |  133 ++++++++++++++++++++++++++++++------------
 1 file changed, 95 insertions(+), 38 deletions(-)