hash: murmur: Add no_sanitize to make ubsan happy

weltling · weltling · commit 7cfaafb4c290 · 2020-10-31T13:35:34.000+01:00
It is just a normal array index usage, no alignment is needed.

Signed-off-by: Anatol Belski &lt;ab@php.net&gt;
diff --git a/ext/hash/murmur/endianness.h b/ext/hash/murmur/endianness.h
@@ -55,6 +55,9 @@ FORCE_INLINE uint64_t BSWAP64(uint64_t u)
 }
 #endif
 
+#ifdef __clang__
+__attribute__((no_sanitize("alignment")))
+#endif
 FORCE_INLINE uint32_t getblock32 ( const uint32_t * const p, const int i)
 {
   if (IsBigEndian()) {
@@ -64,6 +67,9 @@ FORCE_INLINE uint32_t getblock32 ( const uint32_t * const p, const int i)
   }
 }
 
+#ifdef __clang__
+__attribute__((no_sanitize("alignment")))
+#endif
 FORCE_INLINE uint64_t getblock64 ( const uint64_t * const p, const int i)
 {
   if (IsBigEndian()) {

Original file line number	Diff line number	Diff line change
`@@ -55,6 +55,9 @@ FORCE_INLINE uint64_t BSWAP64(uint64_t u)`
`55`	`55`	`}`
`56`	`56`	`#endif`
`57`	`57`
	`58`	`+#ifdef __clang__`
	`59`	`+__attribute__((no_sanitize("alignment")))`
	`60`	`+#endif`
`58`	`61`	`FORCE_INLINE uint32_t getblock32 ( const uint32_t * const p, const int i)`
`59`	`62`	`{`
`60`	`63`	`if (IsBigEndian()) {`
`@@ -64,6 +67,9 @@ FORCE_INLINE uint32_t getblock32 ( const uint32_t * const p, const int i)`
`64`	`67`	`}`
`65`	`68`	`}`
`66`	`69`
	`70`	`+#ifdef __clang__`
	`71`	`+__attribute__((no_sanitize("alignment")))`
	`72`	`+#endif`
`67`	`73`	`FORCE_INLINE uint64_t getblock64 ( const uint64_t * const p, const int i)`
`68`	`74`	`{`
`69`	`75`	`if (IsBigEndian()) {`