Hi.

Sending patch with snprintf.

Also note that this patch needs geoip-1.4.7 (if nginx is compiled with ipv6
support).

Best regards.

On Saturday 11 of June 2011 13:10:24 you wrote:
> 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