search

chrislim2888 / IP2Location-C-Library

IP2Location C library enables the user to find the country, region, city, coordinates, zip code, time zone, ISP, domain name, connection type, area code, weather station code, weather station name, mobile, usage types, etc that any IP address or hostname originates from.
https://www.ip2location.com
MIT License
61 stars 30 forks source link

Possible memory leak #39

Closed Dilvish5 closed 4 years ago

Dilvish5 commented 4 years ago

Hello

I would like to report an issue that seems to cause segmentation fault in apache (with mod_ip2location) and libIP2Location From the coredump i extracted this: IP2Location_free_record (libIP2Location.so.1)#012#6 0x00007fc39bbbdde0 ip2location_post_read_request (mod_ip2location.so)#012#7

I've tried to compile the latest version (8cc0f69), but the issue seems to persist.

Thank you very much for all your work!

ip2location commented 4 years ago

Thanks for reporting, let me investigate the issue.

ip2location commented 4 years ago

Using the last version should install "libIP2Location.so.2" instead of "libIP2Location.so.1". Can you confirmed that you have compiled with the latest version?

Dilvish5 commented 4 years ago

Yes, i just renamed the file to .1 to be sure it's using it instead of the old one.

remicollet commented 4 years ago

Yes, i just renamed the file to .1 to be sure it's using it instead of the old one.

This obviously cannot work.... this suffix is direftly related to the ABI... different number means incompatible change.

You need to rebuild mod_ip2location

Dilvish5 commented 4 years ago

You're right, i've rebuilt mod_ip2location today and could not detect any issues. Thank you!!!