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

[BUG]: Invalid MIDI 1.0 iJack string indexes can crash USB MIDI 2.0 driver when building GTBs #577

Open
AmeNote-Michael opened this issue Mar 1, 2025 · 5 comments
Assignees
Labels
area-usb-driver 💻 Related to the USB MIDI 2.0 driver bug 🐞 Something isn't working fixed-awaiting-public-release 🕙 Fixed in our internal builds. Waiting to make its way to a public release.

Comments

@AmeNote-Michael
Copy link
Collaborator

Windows Version

Canary 27802

Service Installation Method

GitHub

SDK version, if appropriate

Customer Preview 2

Location

Windows MIDI Services Console App

Type of bug

Application Crash

Steps to reproduce

  1. Plug in to available USB port X6 from MIDIPlus (or some other devices)
  2. Use device manager to update driver to USB MIDI 2.0 driver
  3. In console run "midi endpoint monitor"

Expected behavior

Able to select device from list to monitor.

Additional notes

MIDI Console app appears to hang and never present list. You have to kill the console terminal window to exit. In some cases can cause green screen.

This issue was found and reported originally by @AmeNote-MikeKent by email.

@AmeNote-Michael AmeNote-Michael added bug 🐞 Something isn't working needs triage 🚑 Bug needs further investigation and triage labels Mar 1, 2025
@AmeNote-Michael AmeNote-Michael added area-usb-driver 💻 Related to the USB MIDI 2.0 driver and removed needs triage 🚑 Bug needs further investigation and triage labels Mar 1, 2025
@AmeNote-Michael
Copy link
Collaborator Author

Usind debugger it was discovered that the building of group terminal blocks in case of USB MIDI 1.0 device was not incrementing the number of group terminal blocks due to the X6 USB Descriptor having an error in pointing to an invalid String Descriptor.

The driver handled the invalid string descriptor as a warning, but did not properly increment number of group terminal blocks.

@AmeNote-Michael
Copy link
Collaborator Author

Descriptor dump shows invalid string indexes.

DescriptorDump_X6III.txt

@AmeNote-Michael
Copy link
Collaborator Author

@Psychlist1972 resolved in Pull Request #578

@Psychlist1972
Copy link
Contributor

You need to restart the service after changing the driver associated with a device. The service doesn't get a good notification that the driver has changed.

Related #403
Duplicate: #483

@AmeNote-Michael
Copy link
Collaborator Author

@Psychlist1972 this is not same thing - there was a bug in driver. There is a pull request related to this bug.

@Psychlist1972 Psychlist1972 changed the title [BUG]: MIDI Console Application stuck when connected to some USB MIDI 1.0 Devices through new driver [BUG]: Invalid MIDI 1.0 iJack string indexes can crash USB MIDI 2.0 driver when building GTBs Mar 2, 2025
@Psychlist1972 Psychlist1972 added the fixed-awaiting-public-release 🕙 Fixed in our internal builds. Waiting to make its way to a public release. label Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-usb-driver 💻 Related to the USB MIDI 2.0 driver bug 🐞 Something isn't working fixed-awaiting-public-release 🕙 Fixed in our internal builds. Waiting to make its way to a public release.
Projects
None yet
Development

No branches or pull requests

2 participants