On 04/09/2023 22:49, Takashi Iwai wrote:
Yes, that was a bad change. Could you try the fix below?
thanks,
Takashi
-- 8< --
--- a/sound/core/seq/seq_memory.c
+++ b/sound/core/seq/seq_memory.c
@@ -187,8 +187,12 @@ int snd_seq_expand_var_event(const struct snd_seq_event *event, int count, char
err = expand_var_event(event, 0, len, buf, in_kernel);
if (err < 0)
return err;
- if (len != newlen)
- memset(buf + len, 0, newlen - len);
+ if (len != newlen) {
+ if (in_kernel)
+ memset(buf + len, 0, newlen - len);
+ else
+ clear_user((__force void __user *)buf + len, newlen - len);
+ }
return newlen;
}
EXPORT_SYMBOL(snd_seq_expand_var_event);
That patch seems to work fine! Many thanks.
--
Ash Holland, she/her
