asgardius/alsa-utils
1
0
Fork 0
mirror of https://github.com/alsa-project/alsa-utils synced 2024-11-09 17:35:42 +01:00
Code Issues Projects Releases Packages Wiki Activity

aplaymidi: Set event completely for tempo event

Browse source 
After UMP support was added in b399fb8 ev.type setting was inadvertently
dropped in the code path handling tempo meta event.
This is causing tempo meta events to not be handled at all.
Moreover, snd_seq_ev_set_fixed is also missing so MIDI files with
variable events such as SYSEX before the tempo meta event usually are
causing a segfault.

Fixes: b399fb85a9 ("aplaymidi: Add UMP support")
Closes: https://github.com/alsa-project/alsa-utils/issues/241
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
John K. Luebs 2024-01-07 13:33:33 -06:00 committed by Takashi Iwai
parent 827905c002
commit 4cb3e3a7b5
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
seq/aplaymidi/aplaymidi.c
View file

@ -819,6 +819,8 @@ static void play_midi(void)
ev.time.tick = event->tick; ev.time.tick = event->tick;
ev.dest = ports[event->port]; ev.dest = ports[event->port];
if (event->type == SND_SEQ_EVENT_TEMPO) { if (event->type == SND_SEQ_EVENT_TEMPO) {
snd_seq_ev_set_fixed(&ev);
ev.type = event->type;
ev.dest.client = SND_SEQ_CLIENT_SYSTEM; ev.dest.client = SND_SEQ_CLIENT_SYSTEM;
ev.dest.port = SND_SEQ_PORT_SYSTEM_TIMER; ev.dest.port = SND_SEQ_PORT_SYSTEM_TIMER;
ev.data.queue.queue = queue; ev.data.queue.queue = queue;