diff options
author | Takashi Iwai <tiwai@suse.de> | 2016-02-01 12:04:55 +0100 |
---|---|---|
committer | Jiri Slaby <jslaby@suse.cz> | 2016-02-15 17:07:52 +0100 |
commit | d5ba65be238e4856dd783eb57cb8a377c42fa953 (patch) | |
tree | 1b91023e52f18e7f9de567be84217f08d0cff82e /sound | |
parent | 0440139871c0ff467774b1002deaabc8ead76efe (diff) | |
download | lwn-d5ba65be238e4856dd783eb57cb8a377c42fa953.tar.gz lwn-d5ba65be238e4856dd783eb57cb8a377c42fa953.zip |
ALSA: rawmidi: Remove kernel WARNING for NULL user-space buffer check
commit cc85f7a634cfaf9f0713c6aa06d08817424db37a upstream.
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>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/core/rawmidi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/core/rawmidi.c b/sound/core/rawmidi.c index 7b596b5751db..b600dc2a9a40 100644 --- a/sound/core/rawmidi.c +++ b/sound/core/rawmidi.c @@ -1162,7 +1162,7 @@ static long snd_rawmidi_kernel_write1(struct snd_rawmidi_substream *substream, long count1, result; struct snd_rawmidi_runtime *runtime = substream->runtime; - if (snd_BUG_ON(!kernelbuf && !userbuf)) + if (!kernelbuf && !userbuf) return -EINVAL; if (snd_BUG_ON(!runtime->buffer)) return -EINVAL; |