Wireshark-commits: [Wireshark-commits] master-1.10 cf16e8b: Ensure that the input to strtoul ends w

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: Sun, 9 Mar 2014 11:39:54 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cf16e8b142886bb50ff01cc66623c568a0036647
Submitter: Martin Kaiser (wireshark@xxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

cf16e8b by Evan Huus (eapache@xxxxxxxxx):

    Ensure that the input to strtoul ends with a null.
    
    Otherwise it runs past the end of the array into stack memory. Should fix the
    intermittent DVB-CI decryption test suite failures.
    
    Change-Id: Ice17497e661c8579baf3a546efcb5529beda6b49
    Reviewed-on: https://code.wireshark.org/review/559
    Tested-by: Martin Kaiser <wireshark@xxxxxxxxx>
    Reviewed-by: Martin Kaiser <wireshark@xxxxxxxxx>
    Reviewed-on: https://code.wireshark.org/review/566
    

Actions performed:

    from  3cfa7e7   Use &GitVersion; to fix guide creation
    adds  cf16e8b   Ensure that the input to strtoul ends with a null.


Summary of changes:
 epan/dissectors/packet-dvbci.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)