Truncate MIDI SysEx messages after termination byte
Christian Costa
titan.costa at gmail.com
Mon Jan 14 12:03:23 CST 2013
2013/1/14 <Joerg-Cyril.Hoehle at t-systems.com>:
> Hi,
>
> Christian Costa wrote:
>>I took a look at the
>>alsa code and this code simply does not do what it is supposed to.
> I also looked at it today and noted those bogus lines you quote. Needs
> a patch (+ fix memory leak).
I will take a look at it and send a patch unless someone plan to do it.
I will not be able to test with my 2 HW midi devices right now so I need
someone to test it. That would be good to enter a bug report for that.
Johannes, can you do that?
>
> However, Johannes' change is presumably different, as he wants to
> scan the buffer contents for a F7 terminator and ignore subsequent
> bytes. If I were to decide, I'd like to see more supporting evidence.
In the case the first by is F0 and the last one is F7, the code passes
a uninitialized buffer to the alsa function with a wrong terminating
byte F0 inserted which
is even worst because it opens a new sys ex sequence without end.
The Johanness patch prevents this case to happend by making last byte to be F7.
Bye
Christian
More information about the wine-devel
mailing list