Wireshark · Wireshark-commits: [Wireshark-commits] master de9efa1: LLDP : Fix bug in mdi_power_source hf. While it compiles fine, h

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=de9efa1cb91afddb70d1b56a2574fb0def628bd3
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

de9efa1 by Anish Bhatt (anish@xxxxxxxxxxx):

    LLDP : Fix bug in mdi_power_source hf.
    While it compiles fine, hits an assert in add_string :
    failed assertion "hfinfo->type == FT_STRING || hfinfo->type == FT_STRINGZ || hfinfo->type == FT_STRINGZPAD"
    
    Fix takes care of this issue, but seems like we're losing the bitmask info here, is there a better way to do this ?
    
    Change-Id: I7156d734f4b058113730d5dfbf6498e801c36770
    Signed-off-by: Anish Bhatt <anish@xxxxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/1915
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  fbf1873   Make decode_udp_ports() function available for plugins
    adds  de9efa1   LLDP : Fix bug in mdi_power_source hf. While it compiles fine, hits an assert in add_string : failed assertion "hfinfo->type == FT_STRING || hfinfo->type == FT_STRINGZ || hfinfo->type == FT_STRINGZPAD"


Summary of changes:
 epan/dissectors/packet-lldp.c |   18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)