Hello!
On Fri, Sep 09, 2022 at 11:06:05PM +0800, lucas zhang wrote:
> # HG changeset patch
> # User Tong Zhang <zhangtong2...@whu.edu.cn>
> # Date 1662734441 -28800
> # Fri Sep 09 22:40:41 2022 +0800
> # Node ID a1a31f97ae70621282ad04302acdb6c9def306f7
> # Parent ba5cf8f73a2d0a3615565bf9545f3d65216a0530
> Core: fixed potential memory leak in ngx_core_module_create_conf().
>
> When "ngx_array_init" failed and returns a NULL, the object held by
> "ccf" may be leaked.
>
> diff -r ba5cf8f73a2d -r a1a31f97ae70 src/core/nginx.c
> --- a/src/core/nginx.c Thu Sep 08 13:53:49 2022 +0400
> +++ b/src/core/nginx.c Fri Sep 09 22:40:41 2022 +0800
> @@ -1076,6 +1076,7 @@
> if (ngx_array_init(&ccf->env, cycle->pool, 1, sizeof(ngx_str_t))
> != NGX_OK)
> {
> + ngx_pfree(cycle->pool, ccf);
> return NULL;
> }
Pool allocations are freed automatically when the pool is
destroyed, see here for details:
https://nginx.org/en/docs/dev/development_guide.html#pool
Thanks.
--
Maxim Dounin
http://mdounin.ru/
_______________________________________________
nginx-devel mailing list -- nginx-devel@nginx.org
To unsubscribe send an email to nginx-devel-le...@nginx.org
