dangernoodle-io / TaipanMiner / 25304791161

90%

push

github

fix(mining): wrap DPORT self-test caller with SHA peripheral acquire (#329)

mining_run_self_tests() called sha256_hw_dport_self_test() without holding
the SHA lock or enabling the peripheral clock, so digest reads returned
zeros and the abc-vector check failed. The gate then halted mining + stratum
on every WROOM-32 boot.

Wrap the caller (not the callee) — sha256_hw_dport_init() already holds the
lock when it calls self_test, and esp_crypto_sha_aes_lock is non-recursive,
so adding acquire/release inside self_test would deadlock the mining task on
its second-acquire attempt during backend init.

Document the lock-held precondition in the self_test header.

724 of 868 branches covered (83.41%)

Branch coverage included in aggregate %.

1632 of 1678 relevant lines covered (97.26%)

246013.67 hits per line