Wireshark-commits: [Wireshark-commits] master c9f2b9b: DCERPC: workaround that pidl always emits di
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: Tue, 27 Aug 2019 04:02:47 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c9f2b9b3edcc7d99a0eb9b506ff3b5f0101bd1a8
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

c9f2b9b by Ralph Boehme (slow@xxxxxxxxx):

    DCERPC: workaround that pidl always emits dissect_deferred_pointers()
    
    Commit f57cf9e56c90c0c0c724f415eff35821afaaf849 introduced a DISSECTOR_ASSERT()
    that revealed a deficiency in pidl: currently pidl unconditionally adds calls to
    dissect_deferred_pointers() which breaks dissecting any RPC function that
    has only scalar arguments:
    
      Warn Dissector bug, protocol RPCMDSSVC, in packet 51:
      epan/dissectors/packet-dcerpc.c:2940:
      failed assertion "list_ndr_pointer_list"
    
    Bug: 16022
    Change-Id: I9d3522a3e17ef79b9a8a5acb018104ab398a512a
    Reviewed-on: https://code.wireshark.org/review/34364
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  df3141c   DCERPC: update mdssvc DCERPC dissector from upstream Samba
     add  c9f2b9b   DCERPC: workaround that pidl always emits dissect_deferred_pointers()


Summary of changes:
 epan/dissectors/packet-dcerpc.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)