details: https://hg.nginx.org/njs/rev/714fae197d83 branches: changeset: 2220:714fae197d83 user: Dmitry Volyntsev <xei...@nginx.com> date: Mon Oct 16 18:09:37 2023 -0700 description: Modules: fixed items() method of a shared dictionary.
Previously, under a memory pressure the method might return prematurely while still holding the mutex for the shared memory. This fixes #676 issue on Github. diffstat: nginx/ngx_js_shared_dict.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diffs (12 lines): diff -r b49a98886c02 -r 714fae197d83 nginx/ngx_js_shared_dict.c --- a/nginx/ngx_js_shared_dict.c Fri Oct 06 16:52:23 2023 -0700 +++ b/nginx/ngx_js_shared_dict.c Mon Oct 16 18:09:37 2023 -0700 @@ -819,7 +819,7 @@ njs_js_ext_shared_dict_items(njs_vm_t *v rc = njs_vm_array_alloc(vm, kv, 2); if (rc != NJS_OK) { - return NJS_ERROR; + goto fail; } value = njs_vm_array_push(vm, kv); _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org https://mailman.nginx.org/mailman/listinfo/nginx-devel