Commit bed5b60b authored by Lv Ruyi's avatar Lv Ruyi Committed by Steven Rostedt (Google)
Browse files

proc: bootconfig: Add null pointer check 

kzalloc is a memory allocation function which can return NULL when some
internal memory errors happen. It is safer to add null pointer check.

Link: https://lkml.kernel.org/r/20220329104004.2376879-1-lv.ruyi@zte.com.cn



Cc: stable@vger.kernel.org
Fixes: c1a3c360 ("proc: bootconfig: Add /proc/bootconfig to show boot config list")
Acked-by: default avatarMasami Hiramatsu <mhiramat@kernel.org>
Reported-by: default avatarZeal Robot <zealci@zte.com.cn>
Signed-off-by: default avatarLv Ruyi <lv.ruyi@zte.com.cn>
Signed-off-by: default avatarSteven Rostedt (Google) <rostedt@goodmis.org>
parent 84055411

fs/proc/bootconfig.c

+2 −0
Original line number Diff line number Diff line
@@ -32,6 +32,8 @@ static int __init copy_xbc_key_value_list(char *dst, size_t size) 
	int ret = 0;



	key = kzalloc(XBC_KEYLEN_MAX, GFP_KERNEL);

	if (!key)

		return -ENOMEM;



	xbc_for_each_key_value(leaf, val) {

		ret = xbc_node_compose_key(leaf, key, XBC_KEYLEN_MAX);