Welcome! Log In Create A New Profile

Advanced

[njs] Removed check variable type for CLI completions.

Alexander Borisov
May 06, 2021 09:20AM
details: https://hg.nginx.org/njs/rev/637bc81166a6
branches:
changeset: 1636:637bc81166a6
user: Alexander Borisov <alexander.borisov@nginx.com>
date: Thu May 06 16:07:09 2021 +0300
description:
Removed check variable type for CLI completions.

Found by Coverity (CID 1478003).

diffstat:

src/njs_builtin.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r c1829352e49c -r 637bc81166a6 src/njs_builtin.c
--- a/src/njs_builtin.c Thu May 06 16:07:08 2021 +0300
+++ b/src/njs_builtin.c Thu May 06 16:07:09 2021 +0300
@@ -639,7 +639,7 @@ njs_vm_expression_completions(njs_vm_t *
}

var = ((njs_variable_node_t *) node)->variable;
- value = njs_scope_valid_value(vm, var->index);
+ value = njs_scope_value(vm, var->index);

if (!njs_is_object(value)) {
return NULL;
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[njs] Removed check variable type for CLI completions.

Alexander Borisov 162 May 06, 2021 09:20AM



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

Online Users

Guests: 77
Record Number of Users: 6 on February 13, 2018
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready