Wireshark-commits: [Wireshark-commits] master 4d87743: 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 11:28:39 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=4d87743c234c202e810e727d946b578ab38f4e2f
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

4d87743 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>
    

Actions performed:

    from  e754937   WireGuard: implement decryption with PSKs
     add  4d87743   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(-)