Commit 37f1e984 authored by Eugene Teo's avatar Eugene Teo Committed by Linus Torvalds

[PATCH] Fix sequencer missing negative bound check

'int dev' came out of an 'unsigned char *' - as such, it will not get
a negative value. Thanks Valdis.
Signed-off-by: default avatarEugene Teo <eugene.teo@eugeneteo.net>
Cc: Jaroslav Kysela <perex@suse.cz>
Acked-by: default avatarTakashi Iwai <tiwai@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 1ad3dcc0
...@@ -709,11 +709,11 @@ static void seq_local_event(unsigned char *event_rec) ...@@ -709,11 +709,11 @@ static void seq_local_event(unsigned char *event_rec)
static void seq_sysex_message(unsigned char *event_rec) static void seq_sysex_message(unsigned char *event_rec)
{ {
int dev = event_rec[1]; unsigned int dev = event_rec[1];
int i, l = 0; int i, l = 0;
unsigned char *buf = &event_rec[2]; unsigned char *buf = &event_rec[2];
if ((int) dev > max_synthdev) if (dev > max_synthdev)
return; return;
if (!(synth_open_mask & (1 << dev))) if (!(synth_open_mask & (1 << dev)))
return; return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment