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

Pascal Quantin <pascal.quan...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |CONFIRMED
                 CC|                            |pascal.quan...@gmail.com

--- Comment #5 from Pascal Quantin <pascal.quan...@gmail.com> ---
The bug was introduced by Ifcf1854051c7f5dec911331d9cfc809e6b2dd42e and is
systematic on Windows.

Replacing free by g_free (as it was prior to this change) fixes the crash.
What I observe is the opposite of
https://www.wireshark.org/lists/wireshark-dev/201702/msg00118.html

But when looking at GeoIP 1.6.6 source code (the version we package on Windows)
they use malloc so using free should be the right thing to do. That said, we
are using a malloc from a mingw library and using a free from MSVC CRT...

What I can do is:
- build GeoIP 1.6.10 with the current mingw32/64 and see if it helps
- if not, add the GeoIP_free in our GeoIP pre compiled library and give it
another try

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

Reply via email to