[njs] Added error handling for NJS_VMCODE_FUNCTION_COPY instruction.

Dmitry Volyntsev xeioex at nginx.com
Fri Sep 2 00:50:37 UTC 2022


details:   https://hg.nginx.org/njs/rev/a37e4df8f082
branches:  
changeset: 1945:a37e4df8f082
user:      Dmitry Volyntsev <xeioex at nginx.com>
date:      Thu Sep 01 17:49:06 2022 -0700
description:
Added error handling for NJS_VMCODE_FUNCTION_COPY instruction.

This closes #572 issue on Github.

diffstat:

 src/njs_vmcode.c |  4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diffs (14 lines):

diff -r dfe0eeff0568 -r a37e4df8f082 src/njs_vmcode.c
--- a/src/njs_vmcode.c	Wed Aug 31 18:35:58 2022 -0700
+++ b/src/njs_vmcode.c	Thu Sep 01 17:49:06 2022 -0700
@@ -742,6 +742,10 @@ next:
                 fcopy = (njs_vmcode_function_copy_t *) pc;
                 ret = njs_vmcode_function_copy(vm, fcopy->function,
                                                fcopy->retval);
+                if (njs_slow_path(ret != NJS_OK)) {
+                    goto error;
+                }
+
                 break;
 
             case NJS_VMCODE_FUNCTION_FRAME:



More information about the nginx-devel mailing list