details: http://hg.nginx.org/njs/rev/c4035cc63370
branches:
changeset: 614:c4035cc63370
user: Dmitry Volyntsev <xeioex@nginx.com>
date: Thu Sep 27 17:37:55 2018 +0300
description:
Fixed http response and parent getters.

Getters are expected to set resulting value to the provided
argument, not to vm->retval.

diffstat:

nginx/ngx_http_js_module.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diffs (21 lines):

diff -r 5175716b5e80 -r c4035cc63370 nginx/ngx_http_js_module.c
--- a/nginx/ngx_http_js_module.c Thu Sep 27 17:36:38 2018 +0300
+++ b/nginx/ngx_http_js_module.c Thu Sep 27 17:37:55 2018 +0300
@@ -1844,7 +1844,7 @@ ngx_http_js_ext_get_response(njs_vm_t *v

ctx = ngx_http_get_module_ctx(r, ngx_http_js_module);

- njs_vm_retval_set(vm, njs_value_arg(&ctx->args[1]));
+ njs_value_assign(value, njs_value_arg(&ctx->args[1]));

return NJS_OK;
}
@@ -2189,7 +2189,7 @@ ngx_http_js_ext_get_parent(njs_vm_t *vm,
return NJS_ERROR;
}

- njs_vm_retval_set(vm, njs_value_arg(&ctx->args[0]));
+ njs_value_assign(value, njs_value_arg(&ctx->args[0]));

return NJS_OK;
}
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel