Dmitry Volyntsev
May 11, 2023 12:28AM
details: https://hg.nginx.org/njs/rev/a140e71b0fbf
branches:
changeset: 2115:a140e71b0fbf
user: Dmitry Volyntsev <xeioex@nginx.com>
date: Wed May 10 20:50:53 2023 -0700
description:
Fixed memory allocation failure introduced in fc8d1b125cef.

Found by Coverity (CID 1529969).

diffstat:

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

diffs (12 lines):

diff -r 89c821242caf -r a140e71b0fbf src/njs_builtin.c
--- a/src/njs_builtin.c Tue May 09 22:09:13 2023 -0700
+++ b/src/njs_builtin.c Wed May 10 20:50:53 2023 -0700
@@ -756,7 +756,7 @@ njs_object_completions(njs_vm_t *vm, njs

completion->length = (prefix - expression->start) + key.length + 1;
completion->start = njs_mp_alloc(vm->mem_pool, completion->length);
- if (completion == NULL) {
+ if (njs_slow_path(completion->start == NULL)) {
njs_arr_destroy(array);
array = NULL;
goto done;
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[njs] Fixed memory allocation failure introduced in fc8d1b125cef.

Dmitry Volyntsev 288 May 11, 2023 12:28AM



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

Online Users

Guests: 201
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready