aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTakashi Iwai <tiwai@suse.de>2016-02-01 06:04:55 -0500
committerTakashi Iwai <tiwai@suse.de>2016-02-01 06:22:59 -0500
commitcc85f7a634cfaf9f0713c6aa06d08817424db37a (patch)
tree54d8a7ae120cf8c0b455d0a5f38cb30daca52a9d
parent2d1b5c08366acd46c35a2e9aba5d650cb5bf5c19 (diff)
ALSA: rawmidi: Remove kernel WARNING for NULL user-space buffer check
NULL user-space buffer can be passed even in a normal path, thus it's not good to spew a kernel warning with stack trace at each time. Just drop snd_BUG_ON() macro usage there. BugLink: http://lkml.kernel.org/r/CACT4Y+YfVJ3L+q0i-4vyQVyyPD7V=OMX0PWPi29x9Bo3QaBLdw@mail.gmail.com Reported-by: Dmitry Vyukov <dvyukov@google.com> Cc: <stable@vger.kernel.org> Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r--sound/core/rawmidi.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/core/rawmidi.c b/sound/core/rawmidi.c
index a7759846fbaa..f75d1656272c 100644
--- a/sound/core/rawmidi.c
+++ b/sound/core/rawmidi.c
@@ -1178,7 +1178,7 @@ static long snd_rawmidi_kernel_write1(struct snd_rawmidi_substream *substream,
1178 long count1, result; 1178 long count1, result;
1179 struct snd_rawmidi_runtime *runtime = substream->runtime; 1179 struct snd_rawmidi_runtime *runtime = substream->runtime;
1180 1180
1181 if (snd_BUG_ON(!kernelbuf && !userbuf)) 1181 if (!kernelbuf && !userbuf)
1182 return -EINVAL; 1182 return -EINVAL;
1183 if (snd_BUG_ON(!runtime->buffer)) 1183 if (snd_BUG_ON(!runtime->buffer))
1184 return -EINVAL; 1184 return -EINVAL;