diff options
author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2016-03-10 13:21:11 +0100 |
---|---|---|
committer | Jon Mason <jdmason@kudzu.us> | 2016-03-18 01:52:15 +0100 |
commit | 2572c7fb4e4b941af9a0206ac8093d362ae6d371 (patch) | |
tree | 108abf8ced1b14f933e9415f48df1dbb99c2bac6 /drivers/ntb/test/ntb_perf.c | |
parent | ntb: add missing setup of translation window (diff) | |
download | linux-2572c7fb4e4b941af9a0206ac8093d362ae6d371.tar.xz linux-2572c7fb4e4b941af9a0206ac8093d362ae6d371.zip |
ntb: fix possible NULL dereference
kmalloc can fail and we should check for NULL before using the pointer
returned by kmalloc.
Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk>
Acked-by: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Jon Mason <jdmason@kudzu.us>
Diffstat (limited to 'drivers/ntb/test/ntb_perf.c')
-rw-r--r-- | drivers/ntb/test/ntb_perf.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/ntb/test/ntb_perf.c b/drivers/ntb/test/ntb_perf.c index f798beb01ac6..cf19ff07680d 100644 --- a/drivers/ntb/test/ntb_perf.c +++ b/drivers/ntb/test/ntb_perf.c @@ -550,6 +550,8 @@ static ssize_t debugfs_run_read(struct file *filp, char __user *ubuf, return 0; buf = kmalloc(64, GFP_KERNEL); + if (!buf) + return -ENOMEM; out_offset = snprintf(buf, 64, "%d\n", perf->run); ret = simple_read_from_buffer(ubuf, count, offp, buf, out_offset); kfree(buf); |