details:   https://hg.nginx.org/njs/rev/a37e4df8f082
branches:  
changeset: 1945:a37e4df8f082
user:      Dmitry Volyntsev <xei...@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:
_______________________________________________
nginx-devel mailing list -- nginx-devel@nginx.org
To unsubscribe send an email to nginx-devel-le...@nginx.org

Reply via email to