2011/6/11 Gregor Kališnik <gregor@unimatrix-one.org>:
> Hi.
>

Hi Gregor,

> I've created a patch for nginx GeoIP module to also use the GeoIP IPv6
> version. The patch is attached.
>
> Currently it runs on http://www.podnapisi.net and there are no appearant
> problems.
>

Sounds great,

Maybe should change
+ sprintf(temp_mapped, "::FFFF:%s", temp);
to
+ snprintf(temp_mapped, sizeof(temp_mapped), "::FFFF:%s", temp);

In case we get some forged variable as input (e.g. nginx used as
load-balancer for another nginx and not using REMOTE_ADDR but
X-Forwarded-For).

A.

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