Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d2af9b56 authored by Franky Lin's avatar Franky Lin Committed by Kalle Valo
Browse files

brcmfmac: validate user provided data for memdump before copying 



In patch "brcmfmac: add support for sysfs initiated coredump", a new
scenario of brcmf_debug_create_memdump was added in which the user of
the function might not necessarily provide prefix data. Hence the
function should not assume the data is always valid and should perform a
check before copying.

Reviewed-by: default avatarArend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: default avatarFranky Lin <franky.lin@broadcom.com>
Signed-off-by: default avatarArend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
parent 21c5c83c

drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c

+2 −1
Original line number Diff line number Diff line
@@ -40,6 +40,7 @@ int brcmf_debug_create_memdump(struct brcmf_bus *bus, const void *data, 
	if (!dump)

		return -ENOMEM;



	if (data && len > 0)

		memcpy(dump, data, len);

	err = brcmf_bus_get_memdump(bus, dump + len, ramsize);

	if (err) {