Skip to content

Commit

Permalink
msan: notate more variable assignments from assembly code
Browse files Browse the repository at this point in the history
This was missed in 31ba404 because older
versions of clang did not complain about it. But clang-17, at least, does.
  • Loading branch information
theuni committed Mar 29, 2024
1 parent f41c7c8 commit 68c4020
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/scalar_4x64_impl.h
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 68c4020

Please sign in to comment.