tinyusb.git/src/class/video, 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.

revert _end_of_control_descriptor to Revision: 0a45308a296251e4376815b21f89f0038c75a60e

2026-03-09T14:13:59Z

extend video control parsing to include class-specific VC endpoint descriptor

2026-03-09T03:49:37Z

fixed _end_of_control_descriptor logic. wTotalLength does not include Standard Endpoint Descriptor and Class-specific VC Interrupt Endpoint Descriptor,

2026-03-07T14:56:28Z

so fix that _end_of_control_descriptor include Standard Endpoint Descriptor and Class-specific VC Interrupt Endpoint Descriptor. It will also fix _close_vc_itf, _open_vc_itf. _find_desc_entity parsing.

Update src/class/video/video_device.c

2026-03-07T02:46:38Z

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

cleanup

2026-03-06T18:05:12Z

fixed _open_vc_itf parsing

2026-03-06T18:03:50Z

Standard Interface Descriptor (Video Control) Video Control Header Descriptor Video Control Camera Terminal Descriptor Video Control Output Terminal Descriptor Standard Endpoint Descriptor Class-Specific VC Interrupt Endpoint Descriptor Video Control Header Descriptor's wTotalLength = Header Descriptor + Camera Terminal Descriptor + Output Terminal Descriptor _end_of_control_descriptor is Output Terminal Descriptor End the "end" should +7 for _find_desc searchig Standard Endpoint Descriptor

minor update

2026-02-07T11:01:23Z

Merge remote-tracking branch 'tinyusb/master' into copilot/fix-dcd-edpt-xfer-issue

2025-11-19T21:05:07Z

Signed-off-by: HiFiPhile

Merge remote-tracking branch 'tinyusb/master' into embedded-ideas/master

2025-11-14T14:10:11Z

Signed-off-by: Mengsk