diff mbox series

pstore/ram: Add check for kstrdup

Message ID 20230614093733.36048-1-jiasheng@iscas.ac.cn (mailing list archive)
State Mainlined
Commit d97038d5ec2062733c1e016caf9baaf68cf64ea1
Headers show
Series pstore/ram: Add check for kstrdup | expand

Commit Message

Jiasheng Jiang June 14, 2023, 9:37 a.m. UTC
Add check for the return value of kstrdup() and return the error
if it fails in order to avoid NULL pointer dereference.

Fixes: e163fdb3f7f8 ("pstore/ram: Regularize prz label allocation lifetime")
Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn>
---
 fs/pstore/ram_core.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Kees Cook June 14, 2023, 7:02 p.m. UTC | #1
On Wed, 14 Jun 2023 17:37:33 +0800, Jiasheng Jiang wrote:
> Add check for the return value of kstrdup() and return the error
> if it fails in order to avoid NULL pointer dereference.
> 
> 

Applied to for-next/pstore, thanks!

[1/1] pstore/ram: Add check for kstrdup
      https://git.kernel.org/kees/c/d97038d5ec20
diff mbox series

Patch

diff --git a/fs/pstore/ram_core.c b/fs/pstore/ram_core.c
index 966191d3a5ba..85aaf0fc6d7d 100644
--- a/fs/pstore/ram_core.c
+++ b/fs/pstore/ram_core.c
@@ -599,6 +599,8 @@  struct persistent_ram_zone *persistent_ram_new(phys_addr_t start, size_t size,
 	raw_spin_lock_init(&prz->buffer_lock);
 	prz->flags = flags;
 	prz->label = kstrdup(label, GFP_KERNEL);
+	if (!prz->label)
+		goto err;
 
 	ret = persistent_ram_buffer_map(start, size, prz, memtype);
 	if (ret)