diff options
| author | Zijun Hu <quic_zijuhu@quicinc.com> | 2025-05-07 19:50:27 +0800 |
|---|---|---|
| committer | Andreas Hindborg <a.hindborg@kernel.org> | 2025-05-09 10:11:13 +0200 |
| commit | bbb67d4f85fd00a216fca4ca048e15f8ff6a2195 (patch) | |
| tree | a71ac008a07df14664afcddcccf83f715a944388 | |
| parent | f830edbae247b89228c3e09294151b21e0dc849c (diff) | |
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: Joel Becker <jlbec@evilplan.org>
Reviewed-by: Breno Leitao <leitao@debian.org>
Signed-off-by: Zijun Hu <quic_zijuhu@quicinc.com>
Link: https://lore.kernel.org/r/20250507-fix_configfs-v3-3-fe2d96de8dc4@quicinc.com
Signed-off-by: Andreas Hindborg <a.hindborg@kernel.org>
| -rw-r--r-- | fs/configfs/item.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/configfs/item.c b/fs/configfs/item.c index 254170a82aa3..c378b5cbf87d 100644 --- a/fs/configfs/item.c +++ b/fs/configfs/item.c @@ -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. */ |