diff options
| author | Masahiro Yamada <masahiroy@kernel.org> | 2021-10-01 07:32:44 +0200 |
|---|---|---|
| committer | Masahiro Yamada <masahiroy@kernel.org> | 2021-10-01 10:28:17 +0200 |
| commit | 229d0cfae5b21bfc42525cf43b0b4279243acc4e (patch) | |
| tree | 46de5d2794e9f067e749db5d914beabbc4c02a10 /scripts/kconfig/symbol.c | |
| parent | kconfig: rename a variable in the lexer to a clearer name (diff) | |
| download | linux-229d0cfae5b21bfc42525cf43b0b4279243acc4e.tar.xz linux-229d0cfae5b21bfc42525cf43b0b4279243acc4e.zip | |
kconfig: remove 'const' from the return type of sym_escape_string_value()
sym_escape_string_value() returns a malloc'ed memory, but as
(const char *). So, it must be casted to (void *) when it is free'd.
This is odd.
The return type of sym_escape_string_value() should be (char *).
I exploited that free(NULL) has no effect.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Diffstat (limited to '')
| -rw-r--r-- | scripts/kconfig/symbol.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/scripts/kconfig/symbol.c b/scripts/kconfig/symbol.c index 5844d636d38f..6bf8665a6a0f 100644 --- a/scripts/kconfig/symbol.c +++ b/scripts/kconfig/symbol.c @@ -871,7 +871,8 @@ struct symbol *sym_find(const char *name) return symbol; } -const char *sym_escape_string_value(const char *in) +/* The returned pointer must be freed on the caller side */ +char *sym_escape_string_value(const char *in) { const char *p; size_t reslen; |