Wireshark · Wireshark-commits: [Wireshark-commits] master-2.2 976be71: Fix another leak, which happens with -E occurrence=l.

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=976be718e3ca27fd14c7139732501a67eefedab8
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.2
Repository: wireshark

Commits:

976be71 by Guy Harris (guy@xxxxxxxxxxxx):

    Fix another leak, which happens with -E occurrence=l.
    
    For each occurrence, if there was already an occurrence in the array, we
    were just removing it. not freeing what it pointed to.
    
    While we're at it, expand comments. and always check the array size with
    "!= 0", not "> 0" - the value is unsigned, so they're equivalent, but
    this makes the code more self-consistent.
    
    Change-Id: I538f46b296a7721a39ba4366c2f6269e7e097b0d
    Reviewed-on: https://code.wireshark.org/review/26328
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 9a5217bdd4680e36f2e92838348bd4ea010d28a0)
    Reviewed-on: https://code.wireshark.org/review/26330
    

Actions performed:

    from  e19aba3   GIOP: use wmem packet scope for get_modname_from_repoid()
    adds  976be71   Fix another leak, which happens with -E occurrence=l.


Summary of changes:
 epan/print.c | 24 ++++++++++++++++++++----
 1 file changed, 20 insertions(+), 4 deletions(-)