Skip to content

Commit 88d23cb

Browse files
committed
Modules: fixed memory leak when module loading failed.
1 parent 9f4a02c commit 88d23cb

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

nginx/ngx_js.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -665,6 +665,7 @@ ngx_njs_clone(ngx_js_ctx_t *ctx, ngx_js_loc_conf_t *cf, void *external)
665665

666666
engine = njs_mp_alloc(njs_vm_memory_pool(vm), sizeof(ngx_engine_t));
667667
if (engine == NULL) {
668+
njs_vm_destroy(vm);
668669
return NULL;
669670
}
670671

@@ -677,6 +678,8 @@ ngx_njs_clone(ngx_js_ctx_t *ctx, ngx_js_loc_conf_t *cf, void *external)
677678

678679
ngx_log_error(NGX_LOG_ERR, ctx->log, 0, "js exception: %V", &exception);
679680

681+
njs_vm_destroy(vm);
682+
680683
return NULL;
681684
}
682685

0 commit comments

Comments
 (0)