only use faster SHA-1 code on machines with good libc

Macs don't have byteswap.h or endian.h.  Change conditionals to only
use the faster SHA-1 code on little-endian machines with byteswap.h.

2 files changed, 9 insertions, 8 deletions

diff --git a/include/mincrypt/sha.h b/include/mincrypt/sha.h
index 2bcc522..af63e87 100644
--- a/include/mincrypt/sha.h
+++ b/include/mincrypt/sha.h
@@ -29,7 +29,6 @@
 #define _EMBEDDED_SHA_H_
 
 #include <inttypes.h>
-#include <endian.h>
 
 #ifdef __cplusplus
 extern "C" {
@@ -38,7 +37,7 @@ extern "C" {
 typedef struct SHA_CTX {
     uint64_t count;
     uint32_t state[5];
-#if __BYTE_ORDER == __LITTLE_ENDIAN
+#if defined(HAVE_ENDIAN_H) && defined(HAVE_LITTLE_ENDIAN)
     union {
         uint8_t b[64];
         uint32_t w[16];
diff --git a/sha.c b/sha.c
index 33d1cb3..e089d79 100644
--- a/sha.c
+++ b/sha.c
@@ -25,13 +25,15 @@
 ** ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
 
-#include <byteswap.h>
-#include <endian.h>
-#include <memory.h>
-
 #include "mincrypt/sha.h"
 
-#if __BYTE_ORDER == __LITTLE_ENDIAN
+// Some machines lack byteswap.h and endian.h.  These have to use the
+// slower code, even if they're little-endian.
+
+#if defined(HAVE_ENDIAN_H) && defined(HAVE_LITTLE_ENDIAN)
+
+#include <byteswap.h>
+#include <memory.h>
 
 // This version is about 28% faster than the generic version below,
 // but assumes little-endianness.
@@ -186,7 +188,7 @@ const uint8_t* SHA_final(SHA_CTX* ctx) {
     return ctx->buf.b;
 }
 
-#else   // __BYTE_ORDER == BIG_ENDIAN
+#else   // #if defined(HAVE_ENDIAN_H) && defined(HAVE_LITTLE_ENDIAN)
 
 #define rol(bits, value) (((value) << (bits)) | ((value) >> (32 - (bits))))