dangernoodle-io / breadboard / 25201002743

100%

push

github

fix(bb_mdns): serialize lifecycle teardown to prevent shutdown race (B1-127) (#168)

- bb_mdns_on_disconnect (wifi event) and bb_mdns_shutdown (esp_register_shutdown_handler) both call bb_mdns_lifecycle_stop, which is not atomic — both callers can pass the started=true check before either flips it to false, then race into mdns_free / mdns_service_remove_all
- second caller asserts in xQueueSemaphoreTake when the mdns service mutex is already gone
- triggered every TaipanMiner OTA push reboot on bitaxe-601; diagnosed via the new bb_log_panic coredump backtrace (B1-124)
- adds a dedicated s_lifecycle_mutex serializing both call paths; concurrency stays a platform concern, lifecycle layer remains host-portable
- validated on bench bitaxe-601: pre-fix OTA push -> assert; post-fix OTA push -> reset_reason=software, /api/diag/panic available=false

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

336 of 336 branches covered (100.0%)

Branch coverage included in aggregate %.

527 of 527 relevant lines covered (100.0%)

487.36 hits per line