Wireshark-commits: [Wireshark-commits] master d6ec8aa: Fix some uses of proto_tree_.._ret_[value_ty
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 31 Jul 2020 20:47:29 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d6ec8aa0f53cb6683a7cc9c85fcea6512919df6c
Submitter: "Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

d6ec8aa by Jaap Keuter (jaap.keuter@xxxxxxxxx):

    Fix some uses of proto_tree_.._ret_[value_type]() calls
    
    Detected by Martin Mathieson, some calls to proto_tree_.._ret_[value_type]()
    were made with incorrect field types. This change fixes a few.
    
    Change-Id: I4fb4877ad12a3bcc68ea173b806d908090921df5
    Reviewed-on: https://code.wireshark.org/review/38004
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Reviewed-by: Martin Mathieson <martin.r.mathieson@xxxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  e6469a7   Fixed the usage of proto_tree_add_bytes
     add  d6ec8aa   Fix some uses of proto_tree_.._ret_[value_type]() calls


Summary of changes:
 epan/dissectors/packet-etsi_card_app_toolkit.c |  9 ++++++---
 epan/dissectors/packet-ndps.c                  | 12 ++++++------
 2 files changed, 12 insertions(+), 9 deletions(-)