27.05.2012, 23:45, "Maxim Dounin" <mdounin@mdounin.ru>:

[...]

>
> 2)
>
>>      geoip_city     /usr/share/GeoIP/GeoIPCity.dat   utf8;
>>      geoip_org      /usr/share/GeoIP/GeoIPASNum.dat  utf8;
>
> Модуль geoip - он, конечно, хороший, но пользуется библиотекой
> от MaxMind'а.  А та известна как минимум тем, что падает на
> некорректных данных в базе..  Так что я бы рекомендовал выключить и
> посмотреть, что будет.

Всем патч!

--
br, Denis F. Latypoff.diff -ru nginx-1.3.0/src/http/modules/ngx_http_geoip_module.c nginx-1.3.0.geoip/src/http/modules/ngx_http_geoip_module.c
--- nginx-1.3.0/src/http/modules/ngx_http_geoip_module.c 2012-05-14 09:00:17.000000000 -0500
+++ nginx-1.3.0.geoip/src/http/modules/ngx_http_geoip_module.c 2012-05-27 19:11:03.000000000 -0500
@@ -310,7 +310,14 @@
v->valid = 1;
v->no_cacheable = 0;
v->not_found = 0;
- v->data = (u_char *) val;
+ v->data = ngx_palloc(r->pool, v->len);
+
+ if (v->data == NULL) {
+ return NGX_ERROR;
+ }
+
+ memcpy (v->data, val, v->len);
+ free ((void *) val);

return NGX_OK;

_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru