-
Notifications
You must be signed in to change notification settings - Fork 577
BBC: Sereal-Dclone-0.003 segfaults since perl 5.31.2 #17255
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
Comments
I'm pretty sure it's Sereal Encoder/Decoder not handling the new world order of PERL_OP_PARENT. In #17245 it breaks the linearization of the op tree. Here it breaks processing the op tree in a different way, possibly ending up with a bad pointer in the final sibling. |
This BBC ticket for Sub-Quote is one to which we should pay special attention. Sub-Quote's revdeps include Dancer2, Specio, et. al. And Specio sits atop the DateTime universe. So that means that a lot of CPAN is not currently installable against blead. Thank you very much. |
This appears to be fixed by Sereal::Decoder/Encoder 4.011. The darwin tests in the matrix are both 4.011 (despite one being an older perl), while the failing freebsd and linux tests are all pre 4.011. |
On Mon, 3 Feb 2020, 10:13 James E Keenan, ***@***.***> wrote:
This BBC ticket for Sub-Quote is one to which we should pay special
attention. Sub-Quote's revdeps include Dancer2, Specio, et. al. And Specio
sits atop the DateTime universe. So that means that a lot of CPAN is not
currently installable against blead
I fixed that two days ago. Not sure the commit id right now tho. See
#17526
Yves
… |
Now there are also pass reports for freebsd and linux. |
Thanks Slaven! I will close the ticket then. Cheers! Yves |
Description
t/dclone.t within Sereal-Dclone-0.003 dies with a segmentation fault since 5.31.2:
http://fast-matrix.cpantesters.org/?dist=Sereal-Dclone-0.003
A gdb backtrace shows the following:
I see that there are a couple of other BBCs where Sereal is involved or possibly involved --- however, the stacktrace here looks somewhat different than in #17245.
The text was updated successfully, but these errors were encountered: