Page MenuHomeFreeBSD
Differential D51225

libusb: implement libusb_get_max_alt_packet_size
Needs ReviewPublic
Actions

Authored by aokblast on Jul 9 2025, 10:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 19, 5:04 AM
Unknown Object (File)
Sat, Nov 15, 11:56 PM
Unknown Object (File)
Thu, Nov 13, 9:57 AM
Unknown Object (File)
Tue, Nov 11, 10:35 PM
Unknown Object (File)
Wed, Nov 5, 8:54 PM
Unknown Object (File)
Tue, Nov 4, 8:35 AM
Unknown Object (File)
Wed, Oct 29, 8:19 AM
Unknown Object (File)
Oct 27 2025, 1:20 AM
View All 55 Files
Subscribers
imp
lwhsu
ziaee

Details

Reviewers
bapt
kevans
Group Reviewers
USB
Summary

The libusb has a function to calculate the size from given interface,
alt_setting, endpoint. Implementing it by refactoring the calculating
function for libusb_get_max_iso_packet_size.

Sponsored By: FreeBSD Foundation

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 66244
Build 63127: arc lint + arc unit

Event Timeline

aokblast created this revision.Jul 9 2025, 10:51 AM
Herald added subscribers: ziaee, imp. · View Herald TranscriptJul 9 2025, 10:51 AM
aokblast requested review of this revision.Jul 9 2025, 10:51 AM
Harbormaster completed remote builds in B65298: Diff 158214.Jul 9 2025, 10:51 AM
aokblast added reviewers: bapt, USB, kevans.Jul 9 2025, 10:51 AM
aokblast updated this revision to Diff 158216.Jul 9 2025, 11:36 AM

Add missing parentheses in return

Harbormaster completed remote builds in B65300: Diff 158216.Jul 9 2025, 11:36 AM
aokblast updated this revision to Diff 160314.Aug 14 2025, 3:26 AM

Rebase to main and fix potential error

Harbormaster completed remote builds in B66218: Diff 160314.Aug 14 2025, 3:26 AM
lwhsu added a subscriber: lwhsu.Aug 14 2025, 5:21 PM
lwhsu added inline comments.
lib/libusb/libusb10.c
642

pdconf seems leaked here.

662

for all the early return below, except the if (ret < 0), pdconf needs being free()'d?

aokblast updated this revision to Diff 160364.Aug 14 2025, 6:02 PM

leaks

Harbormaster completed remote builds in B66244: Diff 160364.Aug 14 2025, 6:02 PM
lwhsu added inline comments.Aug 23 2025, 1:32 AM
lib/libusb/libusb.3
206

do you mean multiply the multiplier?

Revision Contents
Changeset List

PathSize
lib/
libusb/
11 lines
7 lines
157 lines

Diff 160364

View Options

lib/libusb/libusb.h

Loading...
View Options

lib/libusb/libusb.3

Loading...
View Options

lib/libusb/libusb10.c

Loading...