Wireshark-commits: [Wireshark-commits] master 8c211d5: epan: Check for valid cf->edt before free.

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: Wed, 18 Nov 2015 07:18:30 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8c211d51ef66b64c708f7ad079e7e7e913ac9f14
Submitter: Stig Bj?rlykke (stig@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

8c211d5 by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    epan: Check for valid cf->edt before free.
    
    Added a check for a valid cf->edt before calling epan_dissect_free(),
    as this will give an unintended assert.
    
    This is related to da71ccbf778cca5d9c48e010597570a40fd190c0.
    
    Change-Id: I7f7ceb1b25cfa2400063947f674df10ed6a93e9d
    Reviewed-on: https://code.wireshark.org/review/11934
    Reviewed-by: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    

Actions performed:

    from  2cbb583   [docsis->type29ucd] Update to DOCSIS 2.0
    adds  8c211d5   epan: Check for valid cf->edt before free.


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