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.

Sign up for GitHub

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

Jump to bottom

usbd-ccid: panicked at 'assertion failed: packet.len() <= self.long_packet_missing' #150

Closed
sosthene-nitrokey opened this issue Jan 11, 2023 · 3 comments
Closed

usbd-ccid: panicked at 'assertion failed: packet.len() <= self.long_packet_missing' #150

sosthene-nitrokey opened this issue Jan 11, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@sosthene-nitrokey
Copy link
Collaborator

sosthene-nitrokey commented Jan 11, 2023 •
edited
Loading

This bug can be reached by running the example from Nitrokey/opcard-rs#105 and running the gnuk test suite.

Panic location:

thread '<unnamed>' panicked at 'assertion failed: packet.len() <= self.long_packet_missing', /home/soso-nitrokey/.cache/cargo/git/checkouts/nitrokey-3-firmware-0614bf7d47ce1b5c/c02188a/components/usbd-ccid/src/pipe.rs:183:13
@sosthene-nitrokey sosthene-nitrokey added the bug Something isn't working label Jan 11, 2023
@sosthene-nitrokey
Copy link
Collaborator Author

The gnuk test suite hangs for around 1min40s (weird value), and it looks like after that the test suite times out and tries the next test, without any cleanup, so the error is probably somewhere else.

sosthene-nitrokey added a commit that referenced this issue Jan 13, 2023
@sosthene-nitrokey
Fix #150
d56407b 
When using highspeed usb, some calculations worked with usb 1 constants.
This caused packets to be handled improperly.
@sosthene-nitrokey
Copy link
Collaborator Author

I think I got it!

@sosthene-nitrokey sosthene-nitrokey mentioned this issue Jan 13, 2023
Merged
@szszszsz
Copy link
Member

1min40s

100 seconds, perfectly round :-)

sosthene-nitrokey added a commit to trussed-dev/usbd-ccid that referenced this issue Jan 25, 2023
@sosthene-nitrokey
Fix Nitrokey/nitrokey-3-firmware#150
9d39496 
When using highspeed usb, some calculations worked with usb 1 constants.
This caused packets to be handled improperly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@szszszsz @sosthene-nitrokey