Wireshark-commits: [Wireshark-commits] master-2.0 b4feea7: This is now an FT_STRING elsewhere, so m

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 1 Apr 2017 03:43:01 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b4feea73f26ba8510f570846f8a6bb50801cb927
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

b4feea7 by Guy Harris (guy@xxxxxxxxxxxx):

    This is now an FT_STRING elsewhere, so make it one here.
    
    FT_NONE is wrong for string-valued fields.
    
    Change-Id: I2a44c4542c79daa486382bd5b28f6c37c9c46111
    Reviewed-on: https://code.wireshark.org/review/20827
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit aa82f522c28134463b223bc1eaa5d1fd6cd9d9cd)
    Reviewed-on: https://code.wireshark.org/review/20829
    

Actions performed:

    from  0a30922   Provide better error messages for field type mismatches.
    adds  b4feea7   This is now an FT_STRING elsewhere, so make it one here.


Summary of changes:
 epan/dissectors/packet-btavrcp.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)