Wireshark-commits: [Wireshark-commits] master 5a8e9e3: gsm_gsup: check the current len against the
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, 13 Mar 2019 10:05:03 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5a8e9e35b36d235930cc70b42f356418c9ea0da1
Submitter: "Dario Lombardo <lomato@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

5a8e9e3 by Dario Lombardo (lomato@xxxxxxxxx):

    gsm_gsup: check the current len against the caller length.
    
    When the former is greater than the latter, we have a malformed
    packet. Add an expert info and stop the dissection.
    
    Bug: 15585
    Change-Id: I37e35ed7bfe96fe6896fa5bf3c32bae50a427c60
    Reviewed-on: https://code.wireshark.org/review/32382
    Petri-Dish: Dario Lombardo <lomato@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Oliver Smith <osmith@xxxxxxxxxxx>
    Reviewed-by: Dario Lombardo <lomato@xxxxxxxxx>
    

Actions performed:

    from  bf2c6ad   ZigBee: Decode Tariff Label as unicode
     add  5a8e9e3   gsm_gsup: check the current len against the caller length.


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