-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Segmentation fault on var_dump() of simplehtmldom object #3991
Comments
I got this one:
Not a segfault. HHVM build from master. |
We might be using different versions, I'm using Using "HipHop VM 3.2.0 (dbg)" version. It still seems to be an error. Or you can create an invalid string from code? |
Can you try 3.3.0 or nightly? I had master in dbg and didn't get a segfault. |
Tried 3.3, same results. Core dumped: Segmentation fault. |
Interesting. Could you run it under gdb and obtain a backtrace?
|
This was fixed recently (I think), try building from master after 27ae84e If that fixes it we should probably make sure this gets picked for 3.3.1 |
gdb output for 3.3 is at: https://gist.github.com/andrisi/79c11c0d05c0b979d88d |
The patch I referenced wasn't in 3.3 (it will be in 3.3.1) |
Please re-open if you're still seeing crashes in 3.3.1 or master. |
I have error: Fatal error: String length exceeded 2^31-2 on HipHop VM 3.7.0 using on project in Laravel... |
The fix for this bug involved triggering a fatal error if the maximum size of the output buffer is exceeded. |
Perhaps printing nothing, or just stopping would be a more plesant fix, then throwing a fatal error on a simple var_dump? We shall do some (unreliable) check before every var_dump() to avoid fatal errors? But of course, using var_dump() in a real setting is uncommon. |
Ok, so my app is working now - I had missing storage folders for some caching... but it was really strange what it was doing... |
When loading bigger files using the simplehtmldom library (http://simplehtmldom.sourceforge.net/) and then doing a var_dump on it will cause a Segmentation fault. Just do a "wget -o index.html http://index.hu" and then run the code below.
The text was updated successfully, but these errors were encountered: