From f7f0184ba1358cb8659607d2d0105628cb130e4f Mon Sep 17 00:00:00 2001 From: Cory Fields Date: Wed, 27 Mar 2024 15:18:57 +0000 Subject: [PATCH] msan: notate more variable assignments from assembly code This was missed in 31ba40494428dcbf2eb5eb6f2328eca91b0b0746 because older versions of clang did not complain about it. But clang-17, at least, does. --- src/scalar_4x64_impl.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/scalar_4x64_impl.h b/src/scalar_4x64_impl.h index 835b7fa6f8..82cd957f16 100644 --- a/src/scalar_4x64_impl.h +++ b/src/scalar_4x64_impl.h @@ -814,6 +814,9 @@ static void secp256k1_scalar_mul_512(uint64_t *l8, const secp256k1_scalar *a, co : "+d"(pb) : "S"(l8), "D"(a->d) : "rax", "rbx", "rcx", "r8", "r9", "r10", "r11", "r12", "r13", "r14", "r15", "cc", "memory"); + + SECP256K1_CHECKMEM_MSAN_DEFINE(l8, sizeof(*l8) * 8); + #else /* 160 bit accumulator. */ uint64_t c0 = 0, c1 = 0;