[njs] Fixed misplaced njs_internal_error().

Valentin Bartenev vbart at nginx.com
Thu Nov 8 13:46:07 UTC 2018


details:   http://hg.nginx.org/njs/rev/c5a3ac902bc2
branches:  
changeset: 643:c5a3ac902bc2
user:      Valentin Bartenev <vbart at nginx.com>
date:      Thu Nov 08 16:45:54 2018 +0300
description:
Fixed misplaced njs_internal_error().

diffstat:

 njs/njs_variable.c |  3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diffs (16 lines):

diff -r 6ebf35347eee -r c5a3ac902bc2 njs/njs_variable.c
--- a/njs/njs_variable.c	Wed Nov 07 18:41:29 2018 +0300
+++ b/njs/njs_variable.c	Thu Nov 08 16:45:54 2018 +0300
@@ -83,10 +83,11 @@ njs_builtin_add(njs_vm_t *vm, njs_parser
     ret = nxt_lvlhsh_insert(&scope->variables, &lhq);
 
     if (nxt_fast_path(ret == NXT_OK)) {
-        njs_internal_error(vm, "lvlhsh insert failed");
         return var;
     }
 
+    njs_internal_error(vm, "lvlhsh insert failed");
+
     nxt_mem_cache_free(vm->mem_cache_pool, var->name.start);
     nxt_mem_cache_free(vm->mem_cache_pool, var);
 


More information about the nginx-devel mailing list