4 Mar
2014
4 Mar
'14
11:43 p.m.
You don't have the two patches, on master, the code there is:
default: /* cc */
if(gpv->item->r.record==NULL)
return pv_get_null(msg, param, res);
return pv_geoip_get_strzval(msg, param, res,
gpv->item->r.record->country_code);
Be sure you pick commits:
721ffe3576e8e7891c27f8578eb244a857ede759
56ed48bf48c3d78ff5d214833e09a5759f0b5928
Cheers,
Daniel
On 04/03/14 22:49, Juha Heinanen wrote:
Daniel-Constantin Mierla writes:
--
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
Can you give here what is in your file geoip_pv.c
at line 334? I find
there a return that should not be a reason for crash. Be sure you have
the two patches done lately to the module.
i may have lost what is going on, but i
do have return statement on that
line and that statement is referencing gpv item:
default: /* cc */
/*334/* return pv_geoip_get_strzval(msg, param, res,
gpv->item->r.record->country_code);
if you email me a single patch against 4.1, i can give another try.
on the other hand, it should be very easy for you to make this test too:
$ apt-get install libgeoip1 geoip-database-contrib
and then in config
loadmodule "geoip"
modparam("geoip", "path",
"/usr/share/GeoIP/GeoIPCity.dat")
and the test statement:
$var(tmp) = $gip(src=>cc);
-- juha