commit dca63cfa754503d75c615f5c134d9334f54c132b
author Sam Clegg <sbc@chromium.org> Mon Dec 14 15:54:19 2015 -0800
committer Adam Langley <agl@google.com> Tue Dec 15 18:10:40 2015 +0000
tree ecf61a5475704023b683ced082a389b5c3ff93c3
parent afd565ff9cb4fe1d20f4aa2ee7339c86998aafae

Don't abort in |init_once| if |fcntl| returns ENOSYS

Native Client doesn't support fcntl natively and its default
implemention just returns ENOSYS.

Change-Id: Id8615e2f6f0a75a1140f8efd75afde471ccdf466
Reviewed-on: https://boringssl-review.googlesource.com/6721
Reviewed-by: Adam Langley <agl@google.com>

crypto/rand/urandom.c

diff --git a/crypto/rand/urandom.c b/crypto/rand/urandom.c
index 1cc5260..5bf5c73 100644
--- a/crypto/rand/urandom.c
+++ b/crypto/rand/urandom.c
@@ -83,11 +83,15 @@
 
   int flags = fcntl(fd, F_GETFD);
   if (flags == -1) {
-    abort();
-  }
-  flags |= FD_CLOEXEC;
-  if (fcntl(fd, F_SETFD, flags) == -1) {
-    abort();
+    /* Native Client doesn't implement |fcntl|. */
+    if (errno != ENOSYS) {
+      abort();
+    }
+  } else {
+    flags |= FD_CLOEXEC;
+    if (fcntl(fd, F_SETFD, flags) == -1) {
+      abort();
+    }
   }
   urandom_fd = fd;
 }