On Tue, Oct 13, 2015 at 2:27 AM, Geliang Tang <[email protected]> wrote: > If vmalloc is failed, return -ENOMEM. > > Signed-off-by: Geliang Tang <[email protected]> > --- > fs/pstore/pmsg.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/fs/pstore/pmsg.c b/fs/pstore/pmsg.c > index feb5dd2..5a2f05a 100644 > --- a/fs/pstore/pmsg.c > +++ b/fs/pstore/pmsg.c > @@ -37,6 +37,8 @@ static ssize_t write_pmsg(struct file *file, const char > __user *buf, > if (buffer_size > PMSG_MAX_BOUNCE_BUFFER_SIZE) > buffer_size = PMSG_MAX_BOUNCE_BUFFER_SIZE; > buffer = vmalloc(buffer_size); > + if (!buffer) > + return -ENOMEM; > > mutex_lock(&pmsg_lock); > for (i = 0; i < count; ) { > -- > 1.9.1
I thought I saw this fix come through before? Regardless, thank you! Acked-by: Kees Cook <[email protected]> -Kees -- Kees Cook Chrome OS Security -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
