details: https://hg.nginx.org/njs/rev/c2c8cc00176f branches: changeset: 843:c2c8cc00176f user: Dmitry Volyntsev <xei...@nginx.com> date: Tue Mar 26 15:28:18 2019 +0300 description: Treating null and undefined as empty string for external prop set.
diffstat: njs/njs_object.c | 11 ++++++++--- 1 files changed, 8 insertions(+), 3 deletions(-) diffs (21 lines): diff -r b758915e2406 -r c2c8cc00176f njs/njs_object.c --- a/njs/njs_object.c Tue Mar 26 14:51:03 2019 +0300 +++ b/njs/njs_object.c Tue Mar 26 15:28:18 2019 +0300 @@ -684,9 +684,14 @@ njs_external_property_set(njs_vm_t *vm, pq = (njs_property_query_t *) vm->stash; - ret = njs_vm_value_to_ext_string(vm, &s, setval, 0); - if (nxt_slow_path(ret != NXT_OK)) { - return ret; + if (!njs_is_null_or_undefined(setval)) { + ret = njs_vm_value_to_ext_string(vm, &s, setval, 0); + if (nxt_slow_path(ret != NXT_OK)) { + return ret; + } + + } else { + s = nxt_string_value(""); } *retval = *setval; _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org http://mailman.nginx.org/mailman/listinfo/nginx-devel