Mercurial > hg > nginx

changeset 7214:88aad69eccef

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Geo: optimized configuration parser. If the geo block parser has failed, doing more things is pointless.
author Ruslan Ermilov <ru@nginx.com>
date Wed, 21 Feb 2018 15:50:43 +0300
parents c69c13f10502
children 2dc837d16099
files src/http/modules/ngx_http_geo_module.c src/stream/ngx_stream_geo_module.c
diffstat 2 files changed, 10 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/http/modules/ngx_http_geo_module.c	Wed Feb 21 15:50:42 2018 +0300
+++ b/src/http/modules/ngx_http_geo_module.c	Wed Feb 21 15:50:43 2018 +0300
@@ -461,6 +461,10 @@
 
     *cf = save;
 
+    if (rv != NGX_CONF_OK) {
+        goto failed;
+    }
+
     geo->proxies = ctx.proxies;
     geo->proxy_recursive = ctx.proxy_recursive;
 
@@ -555,7 +559,7 @@
     ngx_destroy_pool(ctx.temp_pool);
     ngx_destroy_pool(pool);
 
-    return rv;
+    return NGX_CONF_OK;
 
 failed:
 
--- a/src/stream/ngx_stream_geo_module.c	Wed Feb 21 15:50:42 2018 +0300
+++ b/src/stream/ngx_stream_geo_module.c	Wed Feb 21 15:50:43 2018 +0300
@@ -431,6 +431,10 @@
 
     *cf = save;
 
+    if (rv != NGX_CONF_OK) {
+        goto failed;
+    }
+
     if (ctx.ranges) {
 
         if (ctx.high.low && !ctx.binary_include) {
@@ -522,7 +526,7 @@
     ngx_destroy_pool(ctx.temp_pool);
     ngx_destroy_pool(pool);
 
-    return rv;
+    return NGX_CONF_OK;
 
 failed: