projects
/
platform
/
kernel
/
linux-starfive.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
7e3ee25
)
misc: fastrpc: check return value of devm_kasprintf()
author
Claudiu Beznea
<claudiu.beznea@microchip.com>
Thu, 15 Jun 2023 10:25:46 +0000
(13:25 +0300)
committer
Greg Kroah-Hartman
<gregkh@linuxfoundation.org>
Wed, 19 Jul 2023 14:21:58 +0000
(16:21 +0200)
[ Upstream commit
af2e19d82a116bc622eea84c9faadd5f7e20bec4
]
devm_kasprintf() returns a pointer to dynamically allocated memory.
Pointer could be NULL in case allocation fails. Check pointer validity.
Identified with coccinelle (kmerr.cocci script).
Fixes:
3abe3ab3cdab
("misc: fastrpc: add secure domain support")
Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Link:
https://lore.kernel.org/r/20230615102546.581899-1-claudiu.beznea@microchip.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/misc/fastrpc.c
patch
|
blob
|
history
diff --git
a/drivers/misc/fastrpc.c
b/drivers/misc/fastrpc.c
index
8b1e866
..
e5cabb9
100644
(file)
--- a/
drivers/misc/fastrpc.c
+++ b/
drivers/misc/fastrpc.c
@@
-2030,6
+2030,9
@@
static int fastrpc_device_register(struct device *dev, struct fastrpc_channel_ct
fdev->miscdev.fops = &fastrpc_fops;
fdev->miscdev.name = devm_kasprintf(dev, GFP_KERNEL, "fastrpc-%s%s",
domain, is_secured ? "-secure" : "");
+ if (!fdev->miscdev.name)
+ return -ENOMEM;
+
err = misc_register(&fdev->miscdev);
if (!err) {
if (is_secured)