From: Richard Levitte <levitte@openssl.org>
Date: Sat, 2 Jan 2016 18:19:00 +0000 (+0100)
Subject: Instead of a local hack, implement SIZE_MAX in numbers.h if it's missing
X-Git-Tag: OpenSSL_1_1_0-pre2~131
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1de1d7689a81f2a3ed3348926e6a31ef79a2bc15;p=openssl

Instead of a local hack, implement SIZE_MAX in numbers.h if it's missing

Reviewed-by: Stephen Henson <steve@openssl.org>
---

diff --git a/include/internal/numbers.h b/include/internal/numbers.h
index 1f79ef7972..da06dab7a6 100644
--- a/include/internal/numbers.h
+++ b/include/internal/numbers.h
@@ -105,5 +105,9 @@
 #  define UINT64_MAX __MAXUINT__(uint64_t)
 # endif
 
+# ifndef SIZE_MAX
+#  define SIZE_MAX __MAXUINT__(size_t)
+# endif
+
 #endif
 
diff --git a/ssl/packet_locl.h b/ssl/packet_locl.h
index 48a5f3d7dc..ed7c1ca129 100644
--- a/ssl/packet_locl.h
+++ b/ssl/packet_locl.h
@@ -65,6 +65,8 @@
 # include <openssl/crypto.h>
 # include <openssl/e_os2.h>
 
+# include "internal/numbers.h"
+
 # ifdef __cplusplus
 extern "C" {
 # endif
@@ -111,13 +113,8 @@ __owur static ossl_inline int PACKET_buf_init(PACKET *pkt, unsigned char *buf,
                                               size_t len)
 {
     /* Sanity check for negative values. */
-#ifdef SIZE_MAX
     if (len > (size_t)(SIZE_MAX / 2))
         return 0;
-#else
-    if (len > ((size_t)2 << (sizeof(size_t) * 8 - 1)))
-        return 0;
-#endif
 
     pkt->curr = buf;
     pkt->remaining = len;