At Wed, 28 Jul 2010 20:41:17 +0400, Kulikov Vasiliy wrote: > > get_user() may fail, if so return -EFAULT. > > Signed-off-by: Kulikov Vasiliy <segooon@xxxxxxxxx> Applied now. Thanks. Takashi > --- > sound/oss/midi_synth.c | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/sound/oss/midi_synth.c b/sound/oss/midi_synth.c > index 3bc7104..3c09374 100644 > --- a/sound/oss/midi_synth.c > +++ b/sound/oss/midi_synth.c > @@ -523,7 +523,9 @@ midi_synth_load_patch(int dev, int format, const char __user *addr, > { > unsigned char data; > > - get_user(*(unsigned char *) &data, (unsigned char __user *) &((addr)[hdr_size + i])); > + if (get_user(data, > + (unsigned char __user *)(addr + hdr_size + i))) > + return -EFAULT; > > eox_seen = (i > 0 && data & 0x80); /* End of sysex */ > > -- > 1.7.0.4 > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html