Date: Wed, 13 Jan 2021 14:28:35 +0100 From: Natanael Copa <ncopa@...inelinux.org> To: musl@...ts.openwall.com Cc: Natanael Copa <ncopa@...inelinux.org> Subject: [PATCH] make realpath replace leading // with a single / On some systems a leading double slash may have special meaning, so POSIX says that "If a pathname begins with two successive <slash> characters, the first component following the leading <slash> characters may be interpreted in an implementation-defined manner" While current musl implementation is technically correct, most other systems' (at least GNU libc, freebsd, openbsd, netbsd macOS) implementations will replace a leading // with a single /. Make musl do the same to avoid surprises. : https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap04.html#tag_04_13 --- src/misc/realpath.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/misc/realpath.c b/src/misc/realpath.c index db8b74dc..414b4741 100644 --- a/src/misc/realpath.c +++ b/src/misc/realpath.c @@ -46,9 +46,6 @@ restart: q=0; output[q++] = '/'; p++; - /* Initial // is special. */ - if (stack[p] == '/' && stack[p+1] != '/') - output[q++] = '/'; continue; } -- 2.30.0
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.