[PATCH] canonicalize: Fix uninitialized 'dest' in realpath_stk for glibc 2.35 [BZ #32577]

Florian Weimer fweimer@redhat.com
Mon Jan 20 14:15:48 GMT 2025


* Nikhil R.:

> Fixes a GCC warning when compiling with the '-Os' optimization flag:
> | canonicalize.c: In function 'realpath_stk':
> | canonicalize.c:424:50: error: 'dest' may be used uninitialized in this function [-Werror=maybe-uninitialized]
> |   424 |   return scratch_buffer_dupfree (rname_buf, dest - rname);
>
> Issue specific to glibc 2.35.
>
> Signed-off-by: Nikhil R <nikhilar2410@gmail.com>
> ---
>  stdlib/canonicalize.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/stdlib/canonicalize.c b/stdlib/canonicalize.c
> index 6caed9e70e..061d3b3062 100644
> --- a/stdlib/canonicalize.c
> +++ b/stdlib/canonicalize.c
> @@ -198,7 +198,7 @@ static char *
>  realpath_stk (const char *name, char *resolved,
>                struct scratch_buffer *rname_buf)
>  {
> -  char *dest;
> +  char *dest = NULL;
>    char const *start;
>    char const *end;
>    int num_links = 0;

We discussed this during the patch review call.  We should backport the
stdlib/canonicalize.c fixes to 2.35 instead.  I'll take care of that.

Thanks,
Florian



More information about the Libc-alpha mailing list