commit cf81b540ce938f43308eb37ab2ab6ee726c8f122
author Brian Smith <brian@briansmith.org> Tue Dec 29 12:02:59 2015 -1000
committer David Benjamin <davidben@google.com> Thu Mar 03 18:01:43 2016 +0000
tree 3200f4cb27a6a46b8bb434ff4f77cf01324d8b0a
parent 9867b7dca287c8e74a5c48f20eec1cc6bc14d670

Remove call to |fprintf| in |CRYPTO_once|.

The |fprintf| dependency is quite heavyweight for small targets. Also,
using |fprintf| on a closed file dsecriptor is undefined behavior, and
there's no way that this code can know whether |stderr| has already
been closed. So, just don't do it.

Change-Id: I1277733afe0649ae1324d11cac84826a1056e308
Reviewed-on: https://boringssl-review.googlesource.com/6812
Reviewed-by: David Benjamin <davidben@chromium.org>
Reviewed-by: David Benjamin <davidben@google.com>

crypto/thread_pthread.c

diff --git a/crypto/thread_pthread.c b/crypto/thread_pthread.c
index 68aaab5..2a1c9f8 100644
--- a/crypto/thread_pthread.c
+++ b/crypto/thread_pthread.c
@@ -17,7 +17,6 @@
 #if !defined(OPENSSL_WINDOWS) && !defined(OPENSSL_NO_THREADS)
 
 #include <pthread.h>
-#include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
@@ -76,8 +75,6 @@
 
 void CRYPTO_once(CRYPTO_once_t *once, void (*init)(void)) {
   if (pthread_once(once, init) != 0) {
-    fprintf(stderr,
-            "pthread_once failed. Did you link against a threading library?\n");
     abort();
   }
 }