https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=12497

Guy Harris <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |CONFIRMED
            Summary|bug in diam_dict.c          |ddict_free() doesn't freeze
                   |(diam_dict.l) in the source |everything
                   |code of wireshark           |
     Ever confirmed|0                           |1
                 OS|Ubuntu                      |All
           Severity|Major                       |Minor

--- Comment #1 from Guy Harris <[email protected]> ---
ddict_free() is calling g_free() on some pointers only if the pointers are
null, not if they're non-null, and thus not freeing what those pointers point
to.

free() and g_free() will do nothing when passed null pointers, so the pointer
check can either be fixed (by removing the !) or simply removed.

As Jeff Morriss points out, ddict_free() is never called, so this bug currently
has no effect.  Lowering the priority as a result.

-- 
You are receiving this mail because:
You are watching all bug changes.
___________________________________________________________________________
Sent via:    Wireshark-bugs mailing list <[email protected]>
Archives:    https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://wireshark.org/mailman/options/wireshark-bugs
             mailto:[email protected]?subject=unsubscribe

Reply via email to