Wireshark-commits: [Wireshark-commits] master cb987e8: Don't leak a file descriptor when changing t
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, 1 Jul 2016 18:36:42 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cb987e86ab39ab2453b52ea4bc50ab51a51063e0
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

cb987e8 by Jeff Morriss (jeff.morriss.ws@xxxxxxxxx):

    Don't leak a file descriptor when changing the SSL debug file name.
    
    I guess people don't change the SSL debug file name that often--this bug has
    been here since 2006...
    
    Change-Id: I3db053dd7e0cb9a9e4ae49a310e956b3c77c99f6
    Reviewed-on: https://code.wireshark.org/review/16236
    Petri-Dish: Jeff Morriss <jeff.morriss.ws@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  f3bd70b   fix missing fields for json, ek and pdml when used with -e fields
    adds  cb987e8   Don't leak a file descriptor when changing the SSL debug file name.


Summary of changes:
 epan/dissectors/packet-ssl-utils.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)