Commit bbb67d4f authored by Zijun Hu's avatar Zijun Hu Committed by Andreas Hindborg
Browse files

configfs: Correct error value returned by API config_item_set_name() 



kvasprintf() failure is often caused by memory allocation which has error
code -ENOMEM, but config_item_set_name() returns -EFAULT for the failure.

Fix by returning -ENOMEM instead of -EFAULT for the failure.

Reviewed-by: default avatarJoel Becker <jlbec@evilplan.org>
Reviewed-by: default avatarBreno Leitao <leitao@debian.org>
Signed-off-by: default avatarZijun Hu <quic_zijuhu@quicinc.com>
Link: https://lore.kernel.org/r/20250507-fix_configfs-v3-3-fe2d96de8dc4@quicinc.com


Signed-off-by: default avatarAndreas Hindborg <a.hindborg@kernel.org>
parent f830edba

fs/configfs/item.c

+1 −1
Original line number Diff line number Diff line
@@ -66,7 +66,7 @@ int config_item_set_name(struct config_item *item, const char *fmt, ...) 
		name = kvasprintf(GFP_KERNEL, fmt, args);

		va_end(args);

		if (!name)

			return -EFAULT;

			return -ENOMEM;

	}



	/* Free the old name, if necessary. */