diff options
author | Kees Cook <keescook@chromium.org> | 2016-11-16 01:29:40 +0100 |
---|---|---|
committer | Kees Cook <keescook@chromium.org> | 2016-11-16 01:34:32 +0100 |
commit | fc46d4e453f50d2b376267f180cae250b54f9fb4 (patch) | |
tree | 2c300b1bb28fe0aba34a6b0b0c2ab1fed8e2b3d5 /fs/pstore | |
parent | pstore: Convert console write to use ->write_buf (diff) | |
download | linux-fc46d4e453f50d2b376267f180cae250b54f9fb4.tar.xz linux-fc46d4e453f50d2b376267f180cae250b54f9fb4.zip |
ramoops: add pdata NULL check to ramoops_probe
This adds a check for a NULL platform data, which should only be possible
if a driver incorrectly sets up a probe request without also having defined
the platform_data structure. This is based on a patch from Geliang Tang.
Signed-off-by: Kees Cook <keescook@chromium.org>
Diffstat (limited to 'fs/pstore')
-rw-r--r-- | fs/pstore/ram.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/fs/pstore/ram.c b/fs/pstore/ram.c index 380222432eff..27c059e1760a 100644 --- a/fs/pstore/ram.c +++ b/fs/pstore/ram.c @@ -734,11 +734,20 @@ static int ramoops_probe(struct platform_device *pdev) goto fail_out; } - /* Only a single ramoops area allowed at a time, so fail extra + /* + * Only a single ramoops area allowed at a time, so fail extra * probes. */ - if (cxt->max_dump_cnt) + if (cxt->max_dump_cnt) { + pr_err("already initialized\n"); goto fail_out; + } + + /* Make sure we didn't get bogus platform data pointer. */ + if (!pdata) { + pr_err("NULL platform data\n"); + goto fail_out; + } if (!pdata->mem_size || (!pdata->record_size && !pdata->console_size && !pdata->ftrace_size && !pdata->pmsg_size)) { |