Wireshark-commits: [Wireshark-commits] master 9015824: dicom: when exporting objects, don't assume

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:31 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=9015824c0ea3ee9ff265ae0db488bd3472d1875c
Submitter: "Guy Harris <gharris@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

9015824 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>
    

Actions performed:

    from  245b638   ieee80211-netmon: fix a typo.
     add  9015824   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(-)