Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README.txt: update number of backends #432

Closed
wants to merge 1 commit into from
Closed

README.txt: update number of backends #432

wants to merge 1 commit into from

Conversation

emaste
Copy link

@emaste emaste commented Jun 8, 2019

FreeBSD support was added in commit 74440e2 as the fifth back-end, but
the list was still introduced as "HIDAPI has four back-ends."

FreeBSD support was added in commit 74440e2 as the fifth back-end, but
the list was still introduced as "HIDAPI has four back-ends."
@emaste
Copy link
Author

emaste commented Jun 8, 2019

Opened libusb/hidapi#13 per #431

@emaste emaste closed this Jun 8, 2019
Megamouse pushed a commit to RPCS3/hidapi that referenced this pull request Jan 6, 2023
- new API function: `struct hid_device_info * hid_get_device_info(hid_device *dev);` to get `hid_device_info` after the device already opened;
- reused existing implementation on Windows and macOS from enumeration routines to have the implementation;
- refactored libusb implementation to have a shared routine for `hid_enumerate` and `hid_get_device_info`;
- refactored hidraw implementation to have a shared routine for `hid_enumerate` and `hid_get_device_info`;

Resolves: signal11#431
Closes: signal11#164
Closes: signal11#163
erikolofsson pushed a commit to Malterlib/hidapi that referenced this pull request Jan 12, 2023
…(#497)

Commit 5c9f147 (signal11#432) replaced a call to strdup with an explicit
memcpy to a buffer on the stack.

However, it incorrectly used the buffer size, instead of the clamped
uevent length, as the argument to memcpy, resulting in reads past the
end of uevent:

Fix this by using uevent_len as the argument to memcpy.

Calling strndupa was considered but abandoned, as it is not standard.

Fixes: 5c9f147 (signal11#432)
Fixes: 4779d63d8760
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant