-
-
Notifications
You must be signed in to change notification settings - Fork 142
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
libxml2: fallback to internal entropy #531
Conversation
b923ac8
to
d1b5c36
Compare
I think it might be cleaner to patch it this way? This is a fairly trivial fix and we should submit it upstream. |
Somehow I cannot fork on their selfhosted gitlab instance... |
It seems like upstream has no interest in such use case: |
You didn't link to the error log, and that maintainer doesn't seem to understand what you're talking about. Better tell him that libxml2 and its caller will simply crash when the Linux kernel doesn't have PRNG support. Anyway, when Debian upgrades theirs to 2.13+, they may receive more bug reports. |
After debating with upstream they added a workaround strictly for Linux returning ENOSYS case: https://gitlab.gnome.org/GNOME/libxml2/-/commit/cd491ac07de361c46450bd98f57c8a5c267fa41a This is enough for our current use case where all reports are from Linux users using a kernel does not implement the |
It looks like they really want to get rid of their internal PRNG. |
Changes
This adds a patch to fallback to use the internal entropy in libxml2 instead of just abort when the system's own
getentropy
call failed. The patch should work for both Windows and Linux, on macOS we are using the macOS native libxml implementation which should be not affected by this.The ubuntu focal build and msys2 Windows Clang build also uses an old version that does not use system entropy generator which is also not affected.
Issues
Fixes #530