Wireshark-commits: [Wireshark-commits] master-2.2 bebd3a1: NBAP: check that DCH ID was initialized
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: Mon, 19 Feb 2018 09:25:58 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bebd3a1f50b0a27738d8d3da5b33c1b392eb7273
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

bebd3a1 by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    NBAP: check that DCH ID was initialized before using it
    
    Bug: 14443
    Change-Id: I4dd72badec3d52b37877f2f846bce56d3e32b111
    Reviewed-on: https://code.wireshark.org/review/25882
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    (cherry picked from commit f87bd6f9789a22bd13fc55ebb1691b763ebfa2ef)
    Conflicts:
    	epan/dissectors/asn1/nbap/nbap.cnf
    	epan/dissectors/packet-nbap.c
    Reviewed-on: https://code.wireshark.org/review/25894
    

Actions performed:

    from  afc780e   DICOM: Prevent infinite loop (dissect_dcm_tag)
    adds  bebd3a1   NBAP: check that DCH ID was initialized before using it


Summary of changes:
 epan/dissectors/asn1/nbap/nbap.cnf |  96 ++++++++++---------
 epan/dissectors/packet-nbap.c      | 184 +++++++++++++++++++------------------
 2 files changed, 150 insertions(+), 130 deletions(-)