Skip to content

Memory protection violation fixed if number of found midi devices is greater than declared in MAX_MIDI_DEVICES.#97

Open
sq8mvy wants to merge 2 commits intog0orx:masterfrom
sq8mvy:master
Open

Memory protection violation fixed if number of found midi devices is greater than declared in MAX_MIDI_DEVICES.#97
sq8mvy wants to merge 2 commits intog0orx:masterfrom
sq8mvy:master

Conversation

@sq8mvy
Copy link
Copy Markdown
Contributor

@sq8mvy sq8mvy commented Apr 5, 2021

Memory protection violation fixed if number of found midi devices is greater than declared in MAX_MIDI_DEVICES.
In the case of more subdevices midi (e.g. virmidi - virtual midi has 16 subdevices) on the list there were empty items in the absence of the subdevice name

sq8mvy and others added 2 commits April 5, 2021 16:51
…greater than declared in MAX_MIDI_DEVICES.

In the case of more subdevices midi (e.g. virmidi - virtual midi has 16 subdevices) on the list there were empty items in the absence of the subdevice name.
@andreasdotorg
Copy link
Copy Markdown
Contributor

Merged to andreasdotorg/linhpsdr. This would be better with an error message on overflow, or even dynamic memory allocation.

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.

2 participants