Wireshark-commits: [Wireshark-commits] master-2.4 1160d43: ssl-utils: fix key log read after EOF
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, 08 Aug 2018 12:04:41 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1160d430a52e110c71441ac669b433bcfee2c656
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master-2.4
Repository: wireshark

Commits:

1160d43 by Peter Wu (peter@xxxxxxxxxxxxx):

    ssl-utils: fix key log read after EOF
    
    C99 requires fgets to fail once the EOF bit is set, glibc 2.28 started
    implementing this behavior. Clear the EOF bit to avoid all future reads
    from failing. Add another error check while at it.
    
    Change-Id: I1c5f7e190426d29e3bf437c443b09092ed8d2d35
    Fixes: v1.99.0-rc1-1080-ga69a63f5d1 ("ssl: fix SSL keylog file live-capture use case")
    Reviewed-on: https://code.wireshark.org/review/28984
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit 4d87743c234c202e810e727d946b578ab38f4e2f)
    Reviewed-on: https://code.wireshark.org/review/29012
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  4104929   Another dictionary fix.
     add  1160d43   ssl-utils: fix key log read after EOF


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