summaryrefslogtreecommitdiffstats
path: root/fs/pstore
diff options
context:
space:
mode:
authorGeliang Tang <geliangtang@gmail.com>2017-04-29 09:45:16 +0800
committerKees Cook <keescook@chromium.org>2017-06-27 17:02:36 -0700
commit077090af337f7d833f82be3d556549512398de66 (patch)
treee2573b974aedaa99bda80a07181050a95d7b3b9f /fs/pstore
parentd3762358a739e7ef63873d6923938def99affa93 (diff)
downloadop-kernel-dev-077090af337f7d833f82be3d556549512398de66.zip
op-kernel-dev-077090af337f7d833f82be3d556549512398de66.tar.gz
pstore: use memdup_user
Use memdup_user() helper instead of open-coding to simplify the code. Signed-off-by: Geliang Tang <geliangtang@gmail.com> Signed-off-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'fs/pstore')
-rw-r--r--fs/pstore/platform.c11
1 files changed, 4 insertions, 7 deletions
diff --git a/fs/pstore/platform.c b/fs/pstore/platform.c
index 96fbff7..1b6e0ff 100644
--- a/fs/pstore/platform.c
+++ b/fs/pstore/platform.c
@@ -653,19 +653,16 @@ static int pstore_write_user_compat(struct pstore_record *record,
if (record->buf)
return -EINVAL;
- record->buf = kmalloc(record->size, GFP_KERNEL);
- if (!record->buf)
- return -ENOMEM;
-
- if (unlikely(copy_from_user(record->buf, buf, record->size))) {
- ret = -EFAULT;
+ record->buf = memdup_user(buf, record->size);
+ if (unlikely(IS_ERR(record->buf))) {
+ ret = PTR_ERR(record->buf);
goto out;
}
ret = record->psi->write(record);
-out:
kfree(record->buf);
+out:
record->buf = NULL;
return unlikely(ret < 0) ? ret : record->size;
OpenPOWER on IntegriCloud