alsa-project / alsa-utils

The Advanced Linux Sound Architecture (ALSA) - utilities
GNU General Public License v2.0
197 stars 136 forks source link

aplaymidi: Set event completely for tempo event #253

Closed jkl1337 closed 9 months ago

jkl1337 commented 9 months ago

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 https://github.com/alsa-project/alsa-utils/issues/241

tiwai commented 9 months ago

Merged now, thanks.