Wireshark-commits: [Wireshark-commits] master 5767118: Free dynamic header fields on exit

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 03 Jun 2018 20:18:16 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=5767118ec9098bf8eeae9c0f7860ed3a48477c79
Submitter: Peter Wu (peter@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

5767118 by Stig Bjørlykke (stig@xxxxxxxxxxxxx):

    Free dynamic header fields on exit
    
    Put routine to free all dynamically registered header fields in the
    UAT reset callback to avoid ASAN report for memory leaks on exit.
    
    Handle duplicated entries without leaking memory.
    
    Call proto_free_deregistered_fields() in proto_cleanup() and move
    this after prefs_cleanup() to free the memory used in UATs.
    
    Change-Id: I96545177b5b23b9c20ad8e7751a0d5621c9ca10f
    Reviewed-on: https://code.wireshark.org/review/27907
    Petri-Dish: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  5c51008   Fix handling of guintvars.
    adds  5767118   Free dynamic header fields on exit


Summary of changes:
 epan/dissectors/asn1/ldap/packet-ldap-template.c |  76 ++++++++++------
 epan/dissectors/packet-http.c                    |  81 ++++++++++-------
 epan/dissectors/packet-imf.c                     |  80 ++++++++++-------
 epan/dissectors/packet-ldap.c                    |  84 ++++++++++-------
 epan/dissectors/packet-lwm2mtlv.c                |  62 +++++++++----
 epan/dissectors/packet-sip.c                     | 100 ++++++++++++---------
 epan/dissectors/packet-udp-nm.c                  | 110 +++++++++++++----------
 epan/epan.c                                      |  15 +++-
 epan/proto.c                                     |   1 +
 9 files changed, 376 insertions(+), 233 deletions(-)