Wireshark-commits: [Wireshark-commits] master-2.2 bd05a81: RPC Netlogon: do not leak memory each ti
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 12 Feb 2018 10:04:24 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=bd05a81bebc14010d82df0a8c70c5f5e2ed06eac
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

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

    RPC Netlogon: do not leak memory each time a hash is generated
    
    Use copy_address_swallow() instead of copy_address().
    When inserting the key in the hash map, copy it in wmem file scope.
    
    Bug: 14407
    Change-Id: Ida524d314c943f480dd0e1bf44fd0ded01aafaeb
    Reviewed-on: https://code.wireshark.org/review/25731
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    (cherry picked from commit d2016c6a1bb9a1688da684825cbda889d95238a6)
    Conflicts:
    	epan/dissectors/packet-dcerpc-netlogon.c
    Reviewed-on: https://code.wireshark.org/review/25747
    

Actions performed:

    from  0a2f323   SIGCOMP: use correct message length
    adds  bd05a81   RPC Netlogon: do not leak memory each time a hash is generated


Summary of changes:
 epan/dissectors/packet-dcerpc-netlogon.c | 29 +++++++++++++++++------------
 1 file changed, 17 insertions(+), 12 deletions(-)