Welcome! Log In Create A New Profile

Advanced

[njs] Added missing njs_vm_destroy() calls in CLI.

Dmitry Volyntsev
March 04, 2021 08:54AM
details: https://hg.nginx.org/njs/rev/52ddc3a050be
branches:
changeset: 1616:52ddc3a050be
user: Dmitry Volyntsev <xeioex@nginx.com>
date: Wed Mar 03 18:28:00 2021 +0000
description:
Added missing njs_vm_destroy() calls in CLI.

diffstat:

src/njs_shell.c | 9 +++++++++
1 files changed, 9 insertions(+), 0 deletions(-)

diffs (40 lines):

diff -r da49b292dcef -r 52ddc3a050be src/njs_shell.c
--- a/src/njs_shell.c Fri Feb 26 17:45:56 2021 +0000
+++ b/src/njs_shell.c Wed Mar 03 18:28:00 2021 +0000
@@ -282,6 +282,7 @@ main(int argc, char **argv)
command.start = (u_char *) opts.command;
command.length = njs_strlen(opts.command);
ret = njs_process_script(&opts, vm_options.external, &command);
+ njs_vm_destroy(vm);
}

} else {
@@ -498,6 +499,8 @@ njs_process_file(njs_opts_t *opts, njs_v
size = sb.st_size;
}

+ vm = NULL;
+
source.length = 0;
source.start = realloc(NULL, size);
if (source.start == NULL) {
@@ -577,6 +580,10 @@ njs_process_file(njs_opts_t *opts, njs_v

done:

+ if (vm != NULL) {
+ njs_vm_destroy(vm);
+ }
+
if (source.start != NULL) {
free(source.start);
}
@@ -938,6 +945,8 @@ njs_interactive_shell(njs_opts_t *opts,
free(line.start);
}

+ njs_vm_destroy(vm);
+
return NJS_OK;
}

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

[njs] Added missing njs_vm_destroy() calls in CLI.

Dmitry Volyntsev 255 March 04, 2021 08:54AM



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

Online Users

Guests: 271
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