Wireshark-commits: [Wireshark-commits] master-3.2 d3a0d05: dicom: when exporting objects, don't ass

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: Thu, 6 Aug 2020 20:37:45 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d3a0d059d1aabb2f366c932ef42e3c364b96a919
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

d3a0d05 by Guy Harris (gharris@xxxxxxxxx):

    dicom: when exporting objects, don't assume we have the AE titles.
    
    In dcm_export_create_object(), don't assume that assoc->ae_calling and
    assoc->ae_called are non-null; if we don't have an A-ASSOCIATE request
    earlier in the capture, which we are not guaranteed to have, the called
    and called AE titles won't be set.
    
    Bug: 16748
    Change-Id: I7d6d22d1c23e28b1f0967a803d0d89609a421712
    Reviewed-on: https://code.wireshark.org/review/38077
    Petri-Dish: Guy Harris <gharris@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <gharris@xxxxxxxxx>
    (cherry picked from commit 476242355716736d07a90d9098810dc1d522ad6b)
    Reviewed-on: https://code.wireshark.org/review/38078
    

Actions performed:

    from  11f78aa   ieee80211-netmon: fix a typo.
     add  d3a0d05   dicom: when exporting objects, don't assume we have the AE titles.


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