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
crypto/rand: nacl support is broken on both 1.4 and tip #9261
Comments
OK. I found out that it's because in my trunk version of sel_ldr, SecureRandom @rsc What should we do here? Should we switch to use NaClSysGetRandomBytes I think the best way for 1.5 is to migrate to the syscall, since we can remove the SRPC However, migrating means we lose the ability to use older sel_ldrs. What do you think? |
I confirm that after I updated native_client to svn revision 13696, nacltest.bash |
I think it is fine to require a newer version of sel_ldr.
|
Or switch to IRT, which has a stable ABI and API. But that's a lot work. |
Patch is ready. It also fixes #9256 for real. The patch is at https://gist.github.com/minux/c7c997669fec1cf391f9, You can use the sel_ldrs in the pepper_canary or pepper_40 version I haven't updated misc/nacl/README yet. WIll do that when sending |
Is this patch waiting for anything? |
I need to update the README. The code is ready.
|
This program (http://play.golang.org/p/0DMMipySJX) runs on playground (Go 1.3)
But fails to run on latest master branch (it tries to open /dev/urandom)
The text was updated successfully, but these errors were encountered: