Opened 5 years ago

Closed 5 years ago

#11533 closed bug (fixed)

KDL General Protection Fault in usb_midi driver on unplug

Reported by: kallisti5 Owned by: mmlr
Priority: normal Milestone: R1
Component: Drivers/USB Version: R1/Development
Keywords: midi_usb_read_callback Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

Unplugging a usb_midi device results in a KDL.

Attachments (4)

IMG_20141129_100051.jpg (963.8 KB ) - added by kallisti5 5 years ago.
SAM_0675.JPG (2.3 MB ) - added by andy-kras 5 years ago.
SAM_0676.JPG (2.5 MB ) - added by andy-kras 5 years ago.
usb_midi_src_for_Masterkey_25_keyboard.zip (61.7 KB ) - added by andy-kras 5 years ago.

Change History (10)

by kallisti5, 5 years ago

Attachment: IMG_20141129_100051.jpg added

comment:1 by kallisti5, 5 years ago

Component: - GeneralDrivers/USB
Keywords: midi_usb_read_callback added
Owner: changed from nobody to mmlr

comment:2 by pulkomandy, 5 years ago

Resolution: fixed
Status: newclosed

Fixed in hrev48750.

comment:3 by andy-kras, 5 years ago

Resolution: fixed
Status: closedreopened

Hello, I have tested new fixed usb_midi driver in hrev48753 x86_gcc2_hybrid with my USB midi keyboard "Acorn Masterkey 25".

There is some progress, but KDL is still issued at keyboard unplugging.

Perhaps, it crashes because my keyboard has no midi-output devices but only midi-input.

I still have to use dirty edited version of usb_midi driver (whith removed output objects).

See attached KDL on hrev48753 and hrev48749, and dirty but working well with my keyboard driver.

Last edited 5 years ago by andy-kras (previous) (diff)

by andy-kras, 5 years ago

Attachment: SAM_0675.JPG added

by andy-kras, 5 years ago

Attachment: SAM_0676.JPG added

comment:4 by pulkomandy, 5 years ago

I fixed the original bug which is the same as your SAM_0676. SAM_0675 is a different crash. I think it would be better to track this in a separate ticket to avoid confusion.

Please open a new ticket with:

  • SAM_0675 (KDL capture). If possible do a better capture, you cut the "message" at the first line of the screen.
  • The modified driver sources
  • The listusb -v output for your device (with a recent revision, I added dumping of the USB MIDI descriptors)

comment:5 by andy-kras, 5 years ago

Ok, new created ticket is #11810.

Version 0, edited 5 years ago by andy-kras (next)

comment:6 by pulkomandy, 5 years ago

Resolution: fixed
Status: reopenedclosed
Note: See TracTickets for help on using tickets.