Wireshark-commits: [Wireshark-commits] master 3ad9768: ssl, dtls: split init/cleanup routines
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: Fri, 3 Jul 2015 23:38:32 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=3ad976896a983178fc42a0794f137983f528cde7
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

3ad9768 by Peter Wu (peter@xxxxxxxxxxxxx):

    ssl,dtls: split init/cleanup routines
    
    Minor functional change: instead of an empty hash table, now the
    ssl_session_hash and ssl_crandom_hash structures point will be set to
    NULL when files are closed.
    
    API change: drop the ssl_keylog_file parameter from ssl_common_init,
    add a new ssl_common_cleanup parameter instead.
    
    Change-Id: I65efe71f8347fe9685359f8ed70cfb9673712421
    Reviewed-on: https://code.wireshark.org/review/9226
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  ba9aa30   Split more init routines into init/cleanup functions
    adds  3ad9768   ssl,dtls: split init/cleanup routines


Summary of changes:
 epan/dissectors/packet-dtls.c      |   11 ++++++++++-
 epan/dissectors/packet-ssl-utils.c |   38 +++++++++++++++---------------------
 epan/dissectors/packet-ssl-utils.h |    5 ++++-
 epan/dissectors/packet-ssl.c       |   25 ++++++++++++++++--------
 4 files changed, 47 insertions(+), 32 deletions(-)