libsecp CI failure [no wallet, libbitcoinkernel] [focal] #28079

issue jonatack opened this issue on July 14, 2023

jonatack commented at 5:51 PM on July 14, 2023: contributor

Is there an existing issue for this?

I have searched the existing issues

Current behaviour

CI failure: https://cirrus-ci.com/task/5453591138271232

Relevant log output

FAIL: tests
===========

test count = 64
random seed = 81af32fd7ab8c9cbc2e62a689f642106
src/modules/ellswift/tests_impl.h:396: test condition failed: secp256k1_memcmp_var(share32_bad, share32a, 32) != 0
FAIL tests (exit status: 134)

jonatack commented at 6:17 PM on July 14, 2023: contributor

Possibly spurious. The CI task passed after re-running it: https://cirrus-ci.com/task/5885691087814656
bitcoin deleted a comment on Jul 15, 2023
maflcko commented at 6:54 AM on July 17, 2023: member

May be good to check if this reproduces with the random seed? cc @jonasnick
jonasnick commented at 9:40 AM on July 17, 2023: contributor

Thanks for reporting. This is an actual bug in the secp256k1 test harness. https://github.com/bitcoin-core/secp256k1/pull/1378
maflcko commented at 11:23 AM on July 17, 2023: member

Depending on how often this happens, we could also bump the subtree here after merge?
real-or-random referenced this in commit b40e2d30b7 on Jul 17, 2023
fanquake closed this on Jul 19, 2023
sidhujag referenced this in commit 381fa93615 on Jul 19, 2023
BlackcoinDev referenced this in commit 84c5416b03 on Feb 5, 2024
bitcoin locked this on Jul 18, 2024

Contributors

jonatack

maflcko

jonasnick

Linked (view graph)

#28093 subtree: update libsecp256k1 to latest master