-
Notifications
You must be signed in to change notification settings - Fork 5
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
Heap corruption #5
Comments
Using libgumbo 0.10.1+dfsg-2.1 (from debian packages) |
There's also a bug for this in the MoarVM tracker: MoarVM/MoarVM#800. I can attest that this is still a killer with the current (git) versions of moar/nqp/rakudo/perl6-gumbo and libgumbo 0.10.1. Now that my scraper has become complex enough I've started hitting this, I am trying to find some kind of workaround before I resort to restarting my whole project in another language. |
@kazcw one of the workarounds that I use is to make things restartable. That is, the script keeps walking until it dies, then it is automatically restarted. Once it starts up, it just continues the work instead of starting from scratch. Another thing that works is if you load a python module using Inline::Python (so instead of using the perl6 gumbo module just use whatever alternative you like in python). It works just fine. |
Maybe you can workaround with fork now? But seriously I wish someone more knowledgeable look at this issue to fix it :( |
Something's up with the CUnions. I'm still trying to get to the root of it. I implemented a workaround in #10. |
Fixes non-deterministic segfaults in anything that non-trivially uses CStruct/CPPStruct/CUnion reprs. Also fixed a separate bug in CUnion: it wasn't caching child objects for attributes that are inlined. Tested to fix the issue in MoarVM MoarVM#800, Skarsnik/perl6-gumbo#5 and perl6 RT 131003. Probably also fixes MoarVM MoarVM#751 and MoarVM#1087
I fixed Moar, but the workaround is of course still necessary for Gumbo to be usable on the current release (mvm 2019.05) |
holy fucking shit!!! I'll try to test this asap… |
Oh, I forgot to reply. But yes, it does work now! |
closing I guess? |
Yep. |
I've filed this RT ticket, but given that this module is completely useless with the bug, I thought I'd create an issue here as well.
The text was updated successfully, but these errors were encountered: