details: https://hg.nginx.org/njs/rev/818ab94037d7
branches:
changeset: 2364:818ab94037d7
user: Dmitry Volyntsev <xei...@nginx.com>
date: Tue Jun 18 23:47:31 2024 -0700
description:
Modules: adding NUL byte at the end of the module body.
Even though QuickJS takes length as an argument, when parsing the code,
it expects NUL byte at the end.
diffstat:
nginx/ngx_js.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diffs (21 lines):
diff -r f358df45922c -r 818ab94037d7 nginx/ngx_js.c
--- a/nginx/ngx_js.c Fri Jun 21 17:58:32 2024 -0700
+++ b/nginx/ngx_js.c Tue Jun 18 23:47:31 2024 -0700
@@ -1841,7 +1841,7 @@ ngx_js_module_read(njs_mp_t *mp, int fd,
text->length = sb.st_size;
- text->start = njs_mp_alloc(mp, text->length);
+ text->start = njs_mp_alloc(mp, text->length + 1);
if (text->start == NULL) {
goto fail;
}
@@ -1852,6 +1852,8 @@ ngx_js_module_read(njs_mp_t *mp, int fd,
goto fail;
}
+ text->start[text->length] = '\0';
+
return NJS_OK;
fail:
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
https://mailman.nginx.org/mailman/listinfo/nginx-devel
