geoip icon indicating copy to clipboard operation
geoip copied to clipboard

Makes sure to `free` a pointer we own

Open alindeman opened this issue 11 years ago • 4 comments

Fixes #18

alindeman avatar Oct 20 '14 14:10 alindeman

@mtodd: This seem OK?

alindeman avatar Oct 21 '14 17:10 alindeman

@alindeman should the change should be applied to rb_geoip_org_look_up et al as well?

mtodd avatar Oct 21 '14 17:10 mtodd

@alindeman should the change should be applied to rb_geoip_org_look_up et al as well?

Yes, somehow I missed these, nice catch. Added a new commit to deal with those.

alindeman avatar Oct 22 '14 14:10 alindeman

@mtodd Can we get these changes merged in?

kaspergrubbe avatar Jul 23 '18 17:07 kaspergrubbe