Wireshark-commits: [Wireshark-commits] master da71ccb: epan: Free cf->edt before free cf->epan when

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Fri, 13 Nov 2015 14:08:12 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=da71ccbf778cca5d9c48e010597570a40fd190c0
Submitter: Stig Bj?rlykke (stig@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

da71ccb by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    epan: Free cf->edt before free cf->epan when redissecting.
    
    This because cf->edt->pi is pointing to data allocated in
    wmem_file_scope and epan_free() will free all data in wmem_file_scope.
    
    In Qt packet_list_clear() we check and use cf->edt, which will give a
    heap-use-after-free error.
    
    Change-Id: I97d532ba976b7bc5fe2995a224ca168edf3da38b
    Reviewed-on: https://code.wireshark.org/review/11803
    Reviewed-by: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    

Actions performed:

    from  f104333   Diameter dictionary updated.
    adds  da71ccb   epan: Free cf->edt before free cf->epan when redissecting.


Summary of changes:
 file.c |    2 ++
 1 file changed, 2 insertions(+)