Welcome! Log In Create A New Profile

Advanced

Re: njs isuse -- ext->set is null

January 13, 2016 12:06PM
Thank you, these and other bugs are fixed in the recent commits.


--
Igor Sysoev
http://nginx.com

On 03 Jan 2016, at 08:18, 洪志道 <hongzhidao@gmail.com> wrote:

> yep, happy new year.
>
> i found anthoer bug.
>
> js_run "
> function foo()
> {
> return 'bug while there is empty statement such ; after return statement';
> }
> ";
>
> i think in the source:
> njs/njs_parser.c/njs_parser_function_lambda: 520
>
> /*
> * There is no function body or the last function body statement is not
> * "return" statement. If function has body then the body->right node is
> * always present and it is a NJS_TOKEN_STATEMENT link node.
> */
> body = parser->node;
>
> if function has body but the last statement is empty such simicolon, the body->right node is also null.
>
>
> 2016-01-03 1:34 GMT+08:00 Igor Sysoev <igor@sysoev.ru>:
> On 02 Jan 2016, at 17:17, 洪志道 <hongzhidao@gmail.com> wrote:
>
>> js_set $bar "
>> $r.error = "this will cause coredump file since ext->set is null";
>> “;
>
> Thank you, I will this after new year vacation.
>
>
> --
> Igor Sysoev
> http://nginx.com
>
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel
>
> _______________________________________________
> nginx-devel mailing list
> nginx-devel@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-devel

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

njs isuse -- ext->set is null

洪志道 491 January 02, 2016 09:18AM

Re: njs isuse -- ext->set is null

Igor Sysoev 473 January 02, 2016 12:36PM

Re: njs isuse -- ext->set is null

洪志道 272 January 03, 2016 12:20AM

Re: njs isuse -- ext->set is null

Igor Sysoev 455 January 13, 2016 12:06PM



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

Online Users

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