Wireshark-commits: [Wireshark-commits] master c7e42be: Catch REPORT_DISSECTOR_BUG() calls in dissec
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: Wed, 28 Oct 2015 00:44:34 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c7e42be2e5baf0a5f351b126cdac5c3d6d9241a0
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

c7e42be by Guy Harris (guy@xxxxxxxxxxxx):

    Catch REPORT_DISSECTOR_BUG() calls in dissector registration routines.
    
    Have epan_init() return a success/failure Boolean indication.  Catch
    exceptions when calling the dissector registration routines and, if we
    get one, report the error and return a failure indication.
    
    If epan_init() fails, quit, but first make sure the reported error is
    displayed.
    
    Change-Id: I0300cbb1f66a5644f857a205235124909d684c50
    Reviewed-on: https://code.wireshark.org/review/11340
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  2831d39   Update Travis to Trusty
    adds  c7e42be   Catch REPORT_DISSECTOR_BUG() calls in dissector registration routines.


Summary of changes:
 dftest.c         |    5 +++--
 epan/epan.c      |   56 +++++++++++++++++++++++++++++++++++++++++-------------
 epan/epan.h      |   14 ++++++++++----
 rawshark.c       |    4 +++-
 tfshark.c        |    4 +++-
 tshark.c         |    4 +++-
 ui/gtk/main.c    |    5 +++--
 wireshark-qt.cpp |    7 +++++--
 8 files changed, 73 insertions(+), 26 deletions(-)