ruby-core

Mailing list archive

[ruby-core:70119] [Ruby trunk - Bug #11395] ruby gets stuck when entropy_avail is 0

From: mame@...
Date: 2015-07-25 08:19:49 UTC
List: ruby-core #70119
Issue #11395 has been updated by Yusuke Endoh.


Nobuyoshi Nakada wrote:
> Does this header exist?

Yes.

~~~~
compiling random.c
random.c: In function 'fill_random_bytes_syscall':
random.c:528:43: error: 'GRND_NONBLOCK' undeclared (first use in this function)
  ret = syscall(SYS_getrandom, seed, size, GRND_NONBLOCK);
                                           ^
random.c:528:43: note: each undeclared identifier is reported only once for each function it appears in
Makefile:369: recipe for target 'random.o' failed
make: *** [random.o] Error 1
~~~~

----------------------------------------
Bug #11395: ruby gets stuck when entropy_avail is 0
https://bugs.ruby-lang.org/issues/11395#change-53550

* Author: Yusuke Endoh
* Status: Assigned
* Priority: Normal
* Assignee: Nobuyoshi Nakada
* ruby -v: ruby 2.3.0dev (2015-07-24 master 51364) [x86_64-linux]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN
----------------------------------------
When `/proc/sys/kernel/random/entropy_avail` is 0, Ruby gets stuck during initialization.

    $ ruby -v
    (stuck)

This is caused by r51182.  I think that the `GRND_NONBLOCK` flag should be set to `getrandom` syscall.

The following patch works for me.  But I have no idea to remove the magic number "1 (= `GRND_NONBLOCK`)".

~~~~diff
diff --git a/random.c b/random.c
index 6452456..9ef89f9 100644
--- a/random.c
+++ b/random.c
@@ -523,11 +523,14 @@ fill_random_bytes_syscall(void *seed, size_t size)
     if (try_syscall) {
 	long ret;
 	errno = 0;
-	ret = syscall(SYS_getrandom, seed, size, 0);
+	ret = syscall(SYS_getrandom, seed, size, 1);
 	if (errno == ENOSYS) {
 	    try_syscall = 0;
 	    return -1;
 	}
+	if (errno == EAGAIN) {
+	    return -1;
+	}
 	if ((size_t)ret == size) return 0;
     }
     return -1;
~~~~

-- 
Yusuke Endoh <mame@ruby-lang.org>



-- 
https://bugs.ruby-lang.org/

In This Thread

Prev Next