-
Notifications
You must be signed in to change notification settings - Fork 8
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
Memory corruption in 0.9.3 #71
Comments
This isn't a bug with SumTypes, it's just being used in a completely invalid way by that library. It's asserting to julia that it can interpret the return type from a I would raise an issue in that library instead. |
Thank you @MasonProtter. My mistake, I meant to file this issue in the Spglib repo, but your input is valuable, thanks. |
Not a problem, happy I could help diagnose. |
We have noticed intermittent crashes since upgrading from Spglib 0.9.2 to 0.9.3. Looks like the only real difference between these versions is the switch from enums to SumTypes: singularitti/Spglib.jl@819b7f6.
Perhaps there is memory corruption when reading an spglib_jll response into a sum type?
Frequently the result is a segfault, but here is one stacktrace we got that seems interpretable:
This appeared on our Github actions CI with Julia 1.9.4, Ubuntu, x86. Full trace here: https://github.com/SunnySuite/Sunny.jl/actions/runs/7629267203/job/20782289111?pr=217
I haven't seen the crashes yet on my Mac. On Linux/x86, reproducing might be as simple as
] add Sunny#spglib_crash
and thenusing Sunny
.The text was updated successfully, but these errors were encountered: