Welcome! Log In Create A New Profile

Advanced

[nginx] Use ngx_pcalloc() in ngx_conf_merge_path_value().

Valentin Bartenev
September 16, 2013 11:08AM
details: http://hg.nginx.org/nginx/rev/7d8770196436
branches:
changeset: 5375:7d8770196436
user: Valentin Bartenev <vbart@nginx.com>
date: Mon Sep 16 18:49:10 2013 +0400
description:
Use ngx_pcalloc() in ngx_conf_merge_path_value().

It initializes the "data" pointer of ngx_path_t that will be checked after
subsequent changes.

diffstat:

src/core/ngx_file.c | 6 +-----
1 files changed, 1 insertions(+), 5 deletions(-)

diffs (23 lines):

diff -r ef3d094bb6d3 -r 7d8770196436 src/core/ngx_file.c
--- a/src/core/ngx_file.c Mon Sep 16 18:33:39 2013 +0400
+++ b/src/core/ngx_file.c Mon Sep 16 18:49:10 2013 +0400
@@ -402,7 +402,7 @@ ngx_conf_merge_path_value(ngx_conf_t *cf
return NGX_CONF_OK;
}

- *path = ngx_palloc(cf->pool, sizeof(ngx_path_t));
+ *path = ngx_pcalloc(cf->pool, sizeof(ngx_path_t));
if (*path == NULL) {
return NGX_CONF_ERROR;
}
@@ -421,10 +421,6 @@ ngx_conf_merge_path_value(ngx_conf_t *cf
+ init->level[1] + (init->level[1] ? 1 : 0)
+ init->level[2] + (init->level[2] ? 1 : 0);

- (*path)->manager = NULL;
- (*path)->loader = NULL;
- (*path)->conf_file = NULL;
-
if (ngx_add_path(cf, path) != NGX_OK) {
return NGX_CONF_ERROR;
}

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

[nginx] Use ngx_pcalloc() in ngx_conf_merge_path_value().

Valentin Bartenev 908 September 16, 2013 11:08AM



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

Online Users

Guests: 159
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready