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 3729a0e commit fa86ffa
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 fa86ffa

Please sign in to comment.