vsha256: Insist that our byte order macros are defined

to avoid errors and confusion as documented in #4226
varnishcache · Nov 18, 2024 · a865812 · a865812
1 parent 117161c
commit a865812
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/lib/libvarnish/vsha256.c b/lib/libvarnish/vsha256.c
@@ -40,6 +40,14 @@
 #  define VBIG_ENDIAN	__DARWIN_BIG_ENDIAN
 #endif
 
+#ifndef VBYTE_ORDER
+# error VBYTE_ORDER not defined
+#endif
+#ifndef VBIG_ENDIAN
+# error VBIG_ENDIAN not defined
+#endif
+
+
 #include <sys/types.h>
 #include <stdint.h>
 #include <string.h>
@@ -50,7 +58,7 @@
 #include "vend.h"
 #include "vsha256.h"
 
-#if defined(VBYTE_ORDER) && VBYTE_ORDER == VBIG_ENDIAN
+#if VBYTE_ORDER == VBIG_ENDIAN
 
 /* Copy a vector of big-endian uint32_t into a vector of bytes */
 #define be32enc_vect(dst, src, len)	\