Wireshark-commits: [Wireshark-commits] master-2.0 0a30922: Provide better error messages for field
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 1 Apr 2017 02:50:53 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0a309226153d8806cba8b18f476ce2bfb2eede84
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

0a30922 by Guy Harris (guy@xxxxxxxxxxxx):

    Provide better error messages for field type mismatches.
    
    Don't just report an assertion failure with a line in proto.c; give the
    name of the field with the wrong type, and a list of the types that are
    valid for the routine in question.
    
    Change-Id: Id3ed7c376fdc72bf6cff69c647833946cfa99ee6
    Reviewed-on: https://code.wireshark.org/review/20824
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 19bb1b128b0e8fdbff0959a00ea062a34fbadfb4)
    Reviewed-on: https://code.wireshark.org/review/20826
    

Actions performed:

    from  4732ebb   Fix endpoints statistics optional filter parsing
    adds  0a30922   Provide better error messages for field type mismatches.


Summary of changes:
 epan/proto.c |   20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)