diff options
| author | Josh Law <objecting@objecting.org> | 2026-03-18 18:59:16 +0300 |
|---|---|---|
| committer | Masami Hiramatsu (Google) <mhiramat@kernel.org> | 2026-03-19 02:44:34 +0300 |
| commit | 68f479de0e013d0c27432240f35bf2a3cc3260f6 (patch) | |
| tree | 41e0a47c56419fd4a7b34970f2e41048c62d9757 | |
| parent | 8f3e79397a980512c2329b3445ebe7258e2d3f01 (diff) | |
| download | linux-68f479de0e013d0c27432240f35bf2a3cc3260f6.tar.xz | |
lib/bootconfig: use size_t for strlen result in xbc_node_match_prefix()
lib/bootconfig.c:198:19: warning: conversion from 'size_t' to 'int'
may change value [-Wconversion]
lib/bootconfig.c:200:33: warning: conversion to '__kernel_size_t'
from 'int' may change the sign of the result [-Wsign-conversion]
strlen() returns size_t but the result was stored in an int. The value
is then passed back to strncmp() which expects size_t, causing a second
sign-conversion warning on the round-trip. Use size_t throughout to
match the API types.
Link: https://lore.kernel.org/all/20260318155919.78168-11-objecting@objecting.org/
Signed-off-by: Josh Law <objecting@objecting.org>
Signed-off-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
| -rw-r--r-- | lib/bootconfig.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/bootconfig.c b/lib/bootconfig.c index 94ae6662531d..9f059bb68e41 100644 --- a/lib/bootconfig.c +++ b/lib/bootconfig.c @@ -195,7 +195,7 @@ static bool __init xbc_node_match_prefix(struct xbc_node *node, const char **prefix) { const char *p = xbc_node_get_data(node); - int len = strlen(p); + size_t len = strlen(p); if (strncmp(*prefix, p, len)) return false; |