]> The Tcpdump Group git mirrors - tcpdump/blobdiff - extract.h
CVE-2017-13687/CHDLC: Improve bounds and length checks.
[tcpdump] / extract.h
index fe69bbccf08fc47b4fb278f822f1d653e99ead89..2ea4ca807d12d5a3ae9d64fdbdee8724390f4e4f 100644 (file)
--- a/extract.h
+++ b/extract.h
  */
 
 /*
- * Macros to extract possibly-unaligned big-endian integral values.
+ * Inline functions or macros to extract possibly-unaligned big-endian
+ * integral values.
  */
+#include "funcattrs.h"
+
+/*
+ * If we have versions of GCC or Clang that support an __attribute__
+ * to say "if we're building with unsigned behavior sanitization,
+ * don't complain about undefined behavior in this function", we
+ * label these functions with that attribute - we *know* it's undefined
+ * in the C standard, but we *also* know it does what we want with
+ * the ISA we're targeting and the compiler we're using.
+ *
+ * For GCC 4.9.0 and later, we use __attribute__((no_sanitize_undefined));
+ * pre-5.0 GCC doesn't have __has_attribute, and I'm not sure whether
+ * GCC or Clang first had __attribute__((no_sanitize(XXX)).
+ *
+ * For Clang, we check for __attribute__((no_sanitize(XXX)) with
+ * __has_attribute, as there are versions of Clang that support
+ * __attribute__((no_sanitize("undefined")) but don't support
+ * __attribute__((no_sanitize_undefined)).
+ *
+ * We define this here, rather than in funcattrs.h, because we
+ * only want it used here, we don't want it to be broadly used.
+ * (Any printer will get this defined, but this should at least
+ * make it harder for people to find.)
+ */
+#if defined(__GNUC__) && ((__GNUC__ * 100 + __GNUC_MINOR__) >= 409)
+#define UNALIGNED_OK   __attribute__((no_sanitize_undefined))
+#elif __has_attribute(no_sanitize)
+#define UNALIGNED_OK   __attribute__((no_sanitize("undefined")))
+#else
+#define UNALIGNED_OK
+#endif
+
 #ifdef LBL_ALIGN
 /*
  * The processor doesn't natively handle unaligned loads.
      defined(__mips) || defined(__mips__))
 
 /*
- * This is a GCC-compatible compiler and we have __attribute__, which
+* This is a GCC-compatible compiler and we have __attribute__, which
  * we assume that mean we have __attribute__((packed)), and this is
  * MIPS or Alpha, which has instructions that can help when doing
  * unaligned loads.
  *
- * Declare packed structures containing a u_int16_t and a u_int32_t,
+ * Declare packed structures containing a uint16_t and a uint32_t,
  * cast the pointer to point to one of those, and fetch through it;
  * the GCC manual doesn't appear to explicitly say that
  * __attribute__((packed)) causes the compiler to generate unaligned-safe
@@ -63,9 +96,9 @@
  * line is aligned, e.g.
  *
  *     #pragma unalign 1
- *     typedef u_int16_t unaligned_u_int16_t;
+ *     typedef uint16_t unaligned_uint16_t;
  *
- * to define unaligned_u_int16_t as a 16-bit unaligned data type.
+ * to define unaligned_uint16_t as a 16-bit unaligned data type.
  * This could be presumably used, in sufficiently recent versions of
  * the compiler, with macros similar to those below.  This would be
  * useful only if that compiler could generate better code for PA-RISC
  * accesses for *specific* items?
  */
 typedef struct {
-       u_int16_t       val;
-} __attribute__((packed)) unaligned_u_int16_t;
+       uint16_t        val;
+} __attribute__((packed)) unaligned_uint16_t;
 
 typedef struct {
-       u_int32_t       val;
-} __attribute__((packed)) unaligned_u_int32_t;
+       uint32_t        val;
+} __attribute__((packed)) unaligned_uint32_t;
 
-static inline u_int16_t
+UNALIGNED_OK static inline uint16_t
 EXTRACT_16BITS(const void *p)
 {
-       return ((u_int16_t)ntohs(((const unaligned_u_int16_t *)(p))->val));
+       return ((uint16_t)ntohs(((const unaligned_uint16_t *)(p))->val));
 }
 
-static inline u_int32_t
+UNALIGNED_OK static inline uint32_t
 EXTRACT_32BITS(const void *p)
 {
-       return ((u_int32_t)ntohl(((const unaligned_u_int32_t *)(p))->val));
+       return ((uint32_t)ntohl(((const unaligned_uint32_t *)(p))->val));
 }
 
-static inline u_int64_t
+UNALIGNED_OK static inline uint64_t
 EXTRACT_64BITS(const void *p)
 {
-       return ((u_int64_t)(((u_int64_t)ntohl(((const unaligned_u_int32_t *)(p) + 0)->val)) << 32 | \
-               ((u_int64_t)ntohl(((const unaligned_u_int32_t *)(p) + 1)->val)) << 0));
+       return ((uint64_t)(((uint64_t)ntohl(((const unaligned_uint32_t *)(p) + 0)->val)) << 32 |
+               ((uint64_t)ntohl(((const unaligned_uint32_t *)(p) + 1)->val)) << 0));
 }
 
 #else /* have to do it a byte at a time */
@@ -116,78 +149,78 @@ EXTRACT_64BITS(const void *p)
  * assemble them.
  */
 #define EXTRACT_16BITS(p) \
-       ((u_int16_t)((u_int16_t)*((const u_int8_t *)(p) + 0) << 8 | \
-                    (u_int16_t)*((const u_int8_t *)(p) + 1)))
+       ((uint16_t)(((uint16_t)(*((const uint8_t *)(p) + 0)) << 8) | \
+                   ((uint16_t)(*((const uint8_t *)(p) + 1)) << 0)))
 #define EXTRACT_32BITS(p) \
-       ((u_int32_t)((u_int32_t)*((const u_int8_t *)(p) + 0) << 24 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 1) << 16 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 2) << 8 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 3)))
+       ((uint32_t)(((uint32_t)(*((const uint8_t *)(p) + 0)) << 24) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 1)) << 16) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 2)) << 8) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 3)) << 0)))
 #define EXTRACT_64BITS(p) \
-       ((u_int64_t)((u_int64_t)*((const u_int8_t *)(p) + 0) << 56 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 1) << 48 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 2) << 40 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 3) << 32 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 4) << 24 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 5) << 16 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 6) << 8 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 7)))
+       ((uint64_t)(((uint64_t)(*((const uint8_t *)(p) + 0)) << 56) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 1)) << 48) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 2)) << 40) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 3)) << 32) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 4)) << 24) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 5)) << 16) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 6)) << 8) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 7)) << 0)))
 #endif /* must special-case unaligned accesses */
 #else /* LBL_ALIGN */
 /*
  * The processor natively handles unaligned loads, so we can just
  * cast the pointer and fetch through it.
  */
-static inline u_int16_t
+static inline uint16_t UNALIGNED_OK
 EXTRACT_16BITS(const void *p)
 {
-       return ((u_int16_t)ntohs(*(const u_int16_t *)(p)));
+       return ((uint16_t)ntohs(*(const uint16_t *)(p)));
 }
 
-static inline u_int32_t
+static inline uint32_t UNALIGNED_OK
 EXTRACT_32BITS(const void *p)
 {
-       return ((u_int32_t)ntohl(*(const u_int32_t *)(p)));
+       return ((uint32_t)ntohl(*(const uint32_t *)(p)));
 }
 
-static inline u_int64_t
+static inline uint64_t UNALIGNED_OK
 EXTRACT_64BITS(const void *p)
 {
-       return ((u_int64_t)(((u_int64_t)ntohl(*((const u_int32_t *)(p) + 0))) << 32 | \
-               ((u_int64_t)ntohl(*((const u_int32_t *)(p) + 1))) << 0));
+       return ((uint64_t)(((uint64_t)ntohl(*((const uint32_t *)(p) + 0))) << 32 |
+               ((uint64_t)ntohl(*((const uint32_t *)(p) + 1))) << 0));
 
 }
 
 #endif /* LBL_ALIGN */
 
 #define EXTRACT_24BITS(p) \
-       ((u_int32_t)((u_int32_t)*((const u_int8_t *)(p) + 0) << 16 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 1) << 8 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 2)))
+       ((uint32_t)(((uint32_t)(*((const uint8_t *)(p) + 0)) << 16) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 1)) << 8) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 2)) << 0)))
 
 #define EXTRACT_40BITS(p) \
-       ((u_int64_t)((u_int64_t)*((const u_int8_t *)(p) + 0) << 32 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 1) << 24 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 2) << 16 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 3) << 8 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 4)))
+       ((uint64_t)(((uint64_t)(*((const uint8_t *)(p) + 0)) << 32) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 1)) << 24) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 2)) << 16) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 3)) << 8) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 4)) << 0)))
 
 #define EXTRACT_48BITS(p) \
-       ((u_int64_t)((u_int64_t)*((const u_int8_t *)(p) + 0) << 40 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 1) << 32 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 2) << 24 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 3) << 16 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 4) << 8 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 5)))
+       ((uint64_t)(((uint64_t)(*((const uint8_t *)(p) + 0)) << 40) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 1)) << 32) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 2)) << 24) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 3)) << 16) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 4)) << 8) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 5)) << 0)))
 
 #define EXTRACT_56BITS(p) \
-       ((u_int64_t)((u_int64_t)*((const u_int8_t *)(p) + 0) << 48 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 1) << 40 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 2) << 32 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 3) << 24 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 4) << 16 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 5) << 8 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 6)))
+       ((uint64_t)(((uint64_t)(*((const uint8_t *)(p) + 0)) << 48) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 1)) << 40) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 2)) << 32) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 3)) << 24) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 4)) << 16) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 5)) << 8) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 6)) << 0)))
 
 /*
  * Macros to extract possibly-unaligned little-endian integral values.
@@ -195,23 +228,50 @@ EXTRACT_64BITS(const void *p)
  */
 #define EXTRACT_LE_8BITS(p) (*(p))
 #define EXTRACT_LE_16BITS(p) \
-       ((u_int16_t)((u_int16_t)*((const u_int8_t *)(p) + 1) << 8 | \
-                    (u_int16_t)*((const u_int8_t *)(p) + 0)))
+       ((uint16_t)(((uint16_t)(*((const uint8_t *)(p) + 1)) << 8) | \
+                   ((uint16_t)(*((const uint8_t *)(p) + 0)) << 0)))
 #define EXTRACT_LE_32BITS(p) \
-       ((u_int32_t)((u_int32_t)*((const u_int8_t *)(p) + 3) << 24 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 2) << 16 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 1) << 8 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 0)))
+       ((uint32_t)(((uint32_t)(*((const uint8_t *)(p) + 3)) << 24) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 2)) << 16) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 1)) << 8) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 0)) << 0)))
 #define EXTRACT_LE_24BITS(p) \
-       ((u_int32_t)((u_int32_t)*((const u_int8_t *)(p) + 2) << 16 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 1) << 8 | \
-                    (u_int32_t)*((const u_int8_t *)(p) + 0)))
+       ((uint32_t)(((uint32_t)(*((const uint8_t *)(p) + 2)) << 16) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 1)) << 8) | \
+                   ((uint32_t)(*((const uint8_t *)(p) + 0)) << 0)))
 #define EXTRACT_LE_64BITS(p) \
-       ((u_int64_t)((u_int64_t)*((const u_int8_t *)(p) + 7) << 56 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 6) << 48 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 5) << 40 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 4) << 32 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 3) << 24 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 2) << 16 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 1) << 8 | \
-                    (u_int64_t)*((const u_int8_t *)(p) + 0)))
+       ((uint64_t)(((uint64_t)(*((const uint8_t *)(p) + 7)) << 56) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 6)) << 48) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 5)) << 40) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 4)) << 32) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 3)) << 24) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 2)) << 16) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 1)) << 8) | \
+                   ((uint64_t)(*((const uint8_t *)(p) + 0)) << 0)))
+
+/*
+ * Macros to check the presence of the values in question.
+ */
+#define ND_TTEST_8BITS(p) ND_TTEST2(*(p), 1)
+#define ND_TCHECK_8BITS(p) ND_TCHECK2(*(p), 1)
+
+#define ND_TTEST_16BITS(p) ND_TTEST2(*(p), 2)
+#define ND_TCHECK_16BITS(p) ND_TCHECK2(*(p), 2)
+
+#define ND_TTEST_24BITS(p) ND_TTEST2(*(p), 3)
+#define ND_TCHECK_24BITS(p) ND_TCHECK2(*(p), 3)
+
+#define ND_TTEST_32BITS(p) ND_TTEST2(*(p), 4)
+#define ND_TCHECK_32BITS(p) ND_TCHECK2(*(p), 4)
+
+#define ND_TTEST_40BITS(p) ND_TTEST2(*(p), 5)
+#define ND_TCHECK_40BITS(p) ND_TCHECK2(*(p), 5)
+
+#define ND_TTEST_48BITS(p) ND_TTEST2(*(p), 6)
+#define ND_TCHECK_48BITS(p) ND_TCHECK2(*(p), 6)
+
+#define ND_TTEST_56BITS(p) ND_TTEST2(*(p), 7)
+#define ND_TCHECK_56BITS(p) ND_TCHECK2(*(p), 7)
+
+#define ND_TTEST_64BITS(p) ND_TTEST2(*(p), 8)
+#define ND_TCHECK_64BITS(p) ND_TCHECK2(*(p), 8)