From b69882d075ca201074de8f4e001af6957f50bc36 Mon Sep 17 00:00:00 2001 From: Dmitry Volyntsev Date: Tue, 25 Nov 2025 15:58:38 -0800 Subject: [PATCH] Fixed missed allocation check in promise code. Found by Coverity (CID 1646931, 1646932). --- src/njs_promise.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/njs_promise.c b/src/njs_promise.c index 86ea334dd..53a8c4e25 100644 --- a/src/njs_promise.c +++ b/src/njs_promise.c @@ -876,6 +876,10 @@ njs_promise_perform_then(njs_vm_t *vm, njs_value_t *value, } else { function = njs_promise_create_function(vm, sizeof(njs_promise_context_t)); + if (njs_slow_path(function == NULL)) { + return NJS_ERROR; + } + function->u.native = njs_promise_reaction_job; if (data->state == NJS_PROMISE_REJECTED) { @@ -967,6 +971,10 @@ njs_promise_prototype_finally(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs, finally = njs_arg(args, nargs, 1); function = njs_promise_create_function(vm, sizeof(njs_promise_context_t)); + if (njs_slow_path(function == NULL)) { + return NJS_ERROR; + } + function->u.native = njs_promise_constructor; njs_set_function(&constructor, function);