Fix __m64i::loadu_si method reading past buffer end

marmeladema · marmeladema · commit 7aba3a20e4a3 · 2021-05-16T19:03:35.000+01:00
Detected with address sanitizer :)
diff --git a/src/x86.rs b/src/x86.rs
@@ -86,7 +86,7 @@ impl Vector for __m64i {
     #[inline]
     #[target_feature(enable = "avx2")]
     unsafe fn loadu_si(a: *const Self) -> Self {
-        __m64i(_mm_loadu_si128(a as *const std::arch::x86_64::__m128i))
+        __m64i(_mm_set_epi64x(0, std::ptr::read_unaligned(a as *const i64)))
     }
 
     #[inline]

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ impl Vector for __m64i {`
`86`	`86`	`#[inline]`
`87`	`87`	`#[target_feature(enable = "avx2")]`
`88`	`88`	`unsafe fn loadu_si(a: *const Self) -> Self {`
`89`		`- __m64i(_mm_loadu_si128(a as *const std::arch::x86_64::__m128i))`
	`89`	`+ __m64i(_mm_set_epi64x(0, std::ptr::read_unaligned(a as *const i64)))`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`#[inline]`