Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

parse hangs #14531

Closed
p5pRT opened this issue Feb 20, 2015 · 6 comments
Closed

parse hangs #14531

p5pRT opened this issue Feb 20, 2015 · 6 comments

Comments

@p5pRT
Copy link

p5pRT commented Feb 20, 2015

Migrated from rt.perl.org#123893 (status was 'resolved')

Searchable as RT123893$

@p5pRT
Copy link
Author

p5pRT commented Feb 20, 2015

From @hvds

I restarted AFL (<http​://lcamtuf.coredump.cx/afl/>) on latest blead, but it quickly finds this and hangs​:

./miniperl -ce '"@​{"'

Bisect shows this was introduced by the recent commit e47d32d​:
  [perl #123712] Fix /$a[/ parsing

  The parser used to read more lines of input when parsing code interpo-
  lated into quote-like operators, under some circumstance.
[...]
  The simplest fix is to modify the function that reads more input,
  namely, lex_next_chunk, and prevent it from reading more lines of
  input from inside a quote-like operator.

Hugo

@p5pRT
Copy link
Author

p5pRT commented Feb 22, 2015

From @cpansprout

Fixed in 0f9d53b.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Feb 22, 2015

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Feb 22, 2015

@cpansprout - Status changed from 'open' to 'pending release'

@p5pRT
Copy link
Author

p5pRT commented Jun 2, 2015

From @khwilliamson

Thank you for submitting this ticket.

The issue should now be resolved with the release today of Perl v5.22, which is available at http​://www.perl.org/get.html
--
Karl Williamson for the Perl 5 team

@p5pRT p5pRT closed this as completed Jun 2, 2015
@p5pRT
Copy link
Author

p5pRT commented Jun 2, 2015

@khwilliamson - Status changed from 'pending release' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant