Welcome! Log In Create A New Profile

Advanced

Re: [nginx] svn commit: r4649 - trunk/src/http/modules

Maxim Dounin
May 28, 2012 10:28AM
Hello!

On Mon, May 28, 2012 at 02:20:04PM +0000, ru@nginx.com wrote:

> Author: ru
> Date: 2012-05-28 14:20:04 +0000 (Mon, 28 May 2012)
> New Revision: 4649
> URL: http://trac.nginx.org/nginx/changeset/4649/nginx
>
> Log:
> Fixed broken build.
>
>
> Modified:
> trunk/src/http/modules/ngx_http_geoip_module.c
>
> Modified: trunk/src/http/modules/ngx_http_geoip_module.c
> ===================================================================
> --- trunk/src/http/modules/ngx_http_geoip_module.c 2012-05-28 13:17:48 UTC (rev 4648)
> +++ trunk/src/http/modules/ngx_http_geoip_module.c 2012-05-28 14:20:04 UTC (rev 4649)
> @@ -310,7 +310,7 @@
> len = ngx_strlen(val);
> v->data = ngx_pnalloc(r->pool, len);
> if (v->data == NULL) {
> - ngx_free(val);
> + ngx_free((void *) val);
> return NGX_ERROR;
> }
>
> @@ -321,7 +321,7 @@
> v->no_cacheable = 0;
> v->not_found = 0;
>
> - ngx_free(val);
> + ngx_free((void *) val);

Correct fix would be to remove "const" from the val. Please try
again.

Maxim Dounin

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[nginx] svn commit: r4649 - trunk/src/http/modules

Anonymous User 1101 May 28, 2012 10:22AM

Re: [nginx] svn commit: r4649 - trunk/src/http/modules

Maxim Dounin 379 May 28, 2012 10:28AM

Re: [nginx] svn commit: r4649 - trunk/src/http/modules

Denis F. Latypoff 393 May 28, 2012 10:46AM

Re: [nginx] svn commit: r4649 - trunk/src/http/modules

Maxim Dounin 422 May 29, 2012 04:40AM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 189
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready