tinyusb.git/src/class/usbtmc/usbtmc_device.c, branch claude/adoring-pasteur-kbaFa

clean up PVS-Studio static-analysis findings for raspberry_pi_pico

2026-06-11T23:34:30Z

Resolves all TinyUSB-owned alerts reported by the CI PVS-Studio job (static_analysis.yml, run with --security-related-issues) on the raspberry_pi_pico board: 0 remaining in src/ and examples/. Genuine fixes: - ncm_device: validate wNdpIndex against sizeof(nth16_t), not the pointer size sizeof(nth16) (4 bytes) — the latter under-checks the NTB header (V568). - tusb: drop the redundant `ff_buf != NULL && ff_bufsize > 0` guard in tu_edpt_stream_init(); the early return already guarantees it (V560). - midi_host: bounds-check idx in tuh_midi_itf_get_info() instead of the always-true `&_midi_host[idx]` pointer (V560). - examples: fully initialize resolutions_per_format / frame_num / interval_ms arrays instead of leaving trailing elements implicitly zero (V1009). False positives suppressed at the cause: - usbd/usbh: hide the weak dcd_deinit()/hcd_deinit() stubs from the analyzer with #ifndef PVS_STUDIO. PVS analyzes one TU at a time and binds the call to the always-false weak stub (it cannot model the linker selecting the port's strong definition), then reports the cleanup loop after TU_ASSERT(...deinit()) as unreachable (V779). False positives suppressed locally (inline //-V or .pvsconfig): - .pvsconfig: V501 (HID descriptor macros), V763 (rhport override), V785 (audio function-index switch), V1044 (hardware poll loops). - inline //-V for config-dependent or intentional constructs: V512, V514-style contiguous clears, V547, V557, V560, V614, V619, V641, V1008, V1037, V1048, V1086. Verified: all examples build for raspberry_pi_pico; ceedling test:all passes (60/60); re-run of the CI-exact PVS invocation reports zero TinyUSB-owned findings.

Fix copilot issues

2025-10-22T18:21:57Z

Signed-off-by: Mengsk

Add is_isr parameter to usbd_edpt_xfer and usbd_edpt_xfer_fifo

2025-10-22T06:12:08Z

- Added bool is_isr parameter to usbd_edpt_xfer() and usbd_edpt_xfer_fifo() wrapper functions - These are called by class drivers to queue USB transfers - Updated all callers to pass false by default (non-ISR context) - Updated audiod_rx_xfer_isr() and audiod_tx_xfer_isr() to pass true (ISR context) - All 21 unit tests pass Co-authored-by: HiFiPhile <4375114+HiFiPhile@users.noreply.github.com>

Update weak callbacks to new syntax

2025-09-17T20:48:59Z

Signed-off-by: HiFiPhile

Update src/class/usbtmc/usbtmc_device.c

2025-07-26T12:01:02Z

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

fix last transfer size

2025-07-11T15:56:56Z

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

typo fix

2025-07-11T15:48:10Z

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Code reformat

2025-06-08T12:31:11Z

Signed-off-by: HiFiPhile

add support for usbtmc vendor-spicific command messages

2025-05-06T14:06:10Z

Fix missing protoype warning, change TUD_EPBUF_TYPE_DEF order (#2889)

2024-11-27T11:28:12Z

* change TUD_EPBUF_TYPE_DEF order * add and fix -Wmissing-prototypes warnings for cmake (skip make)