Wireshark · Wireshark-commits: [Wireshark-commits] master-1.12 ffe97f8: Use packet_scope instead of a stack local

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ffe97f82e09e5124bdcd2504a159ce7923be4b83
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

ffe97f8 by Evan Huus (eapache@xxxxxxxxx):

    Use packet_scope instead of a stack local
    
    As clang pointed out we end up storing a reference to it in a global and (more
    relevantly) pushing that global to a tap which would run after the current frame
    has returned.
    
    Thanks to Alexis for bringing this to my attention.
    
    Change-Id: I3aac43a806d217b0dc8a973f6bb2fa48cdd041bb
    Reviewed-on: https://code.wireshark.org/review/3289
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit a644744fb9502971fee751b8a1e390d255b82d78)
    Reviewed-on: https://code.wireshark.org/review/3310
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  b03c8af   Add wireshark.pc.in to the source distribution (needed for cmake).
    adds  ffe97f8   Use packet_scope instead of a stack local


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