Commit 1180feef authored Feb 29, 2024 by Michael Ellerman Committed by Greg Kroah-Hartman Sep 04, 2024

powerpc/boot: Only free if realloc() succeeds



[ Upstream commit f2d5bccaca3e8c09c9b9c8485375f7bdbb2631d2 ]

simple_realloc() frees the original buffer (ptr) even if the
reallocation failed.

Fix it to behave like standard realloc() and only free the original
buffer if the reallocation succeeded.

Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://msgid.link/20240229115149.749264-1-mpe@ellerman.id.au


Signed-off-by: Sasha Levin <sashal@kernel.org>

parent cd146e31

arch/powerpc/boot/simple_alloc.c

+3 −2

Original line number	Diff line number	Diff line
		@@ -114,10 +114,11 @@ static void simple_realloc(void ptr, unsigned long size)
		return ptr;

		new = simple_malloc(size);
		if (new)
		if (new) {
		memcpy(new, ptr, p->size);

		simple_free(ptr);
		}

		return new;
		}