dangernoodle-io / breadboard / 28749353737

100%

push

github

refactor: adopt bb_strlcpy at class-B named-const copy sites (bb_str judgment) (#736)

- convert remaining strncpy(dst, src, N-1) + dst[N-1]='\0' sites to bb_strlcpy(dst, src, sizeof(dst)) across bb_event, bb_event_routes, bb_ota_check, bb_openapi, bb_http_client (host), bb_pool, bb_ring, bb_pub, bb_sink_http, bb_ota_boot, bb_board (host+espidf), bb_wifi (host), bb_prov, bb_diag_panic, bb_http (host)
- also convert equivalent-pattern sites using a runtime size parameter (out/out_size, buf/len) instead of a named literal constant, since these are the same defect class
- special-case bb_sink_http's heap-allocated tmp buffer: pass HEADERS_BUF_MAX directly, not sizeof(tmp)
- special-case bb_pub's tags[t] field: sizeof(src->tags[t]) is BB_PUB_TAG_MAX + 1, intentionally correct (was strncpy(..., BB_PUB_TAG_MAX) with no -1)
- wire bb_str into bb_event, bb_event_routes, bb_pool, bb_ring, bb_ota_boot component CMakeLists (PRIV_REQUIRES); other touched components already had it from #733/#735
- leave platform/espidf/bb_wifi/bb_wifi.c (class-E raw SSID copy) and platform/arduino/bb_http/bb_http_arduino.cpp (deferred, B1-634) untouched per exclusion list

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

2866 of 2866 branches covered (100.0%)

Branch coverage included in aggregate %.

4 of 4 new or added lines in 3 files covered. (100.0%)

4707 of 4707 relevant lines covered (100.0%)

11152.33 hits per line