ash: do not split the result of tilde expansion
authorRon Yorston <rmy@tigress.co.uk>
Mon, 16 Mar 2015 14:14:11 +0000 (14:14 +0000)
committerDenys Vlasenko <vda.linux@googlemail.com>
Sun, 22 Mar 2015 15:55:44 +0000 (16:55 +0100)
A tilde expansion generates a valid pathname. Splitting it using IFS
either leaves it unchanged or changes it to something unintended.

Example:
   IFS=m HOME=/tmp; printf "%s\n" ~

Based on this commit authored by Jilles Tjoelker:

   http://git.kernel.org/cgit/utils/dash/dash.git/commit/?id=834629283f6c629a4da05ef60bae9445c954a19a

Signed-off-by: Ron Yorston <rmy@tigress.co.uk>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c

index 0f9f73ec369825108b1040ed45d4956f8cb53db6..b568013b425585973519dd08d7f834685a501f5e 100644 (file)
@@ -5791,7 +5791,6 @@ exptilde(char *startp, char *p, int flags)
        struct passwd *pw;
        const char *home;
        int quotes = flags & (EXP_FULL | EXP_CASE | EXP_REDIR);
-       int startloc;
 
        name = p + 1;
 
@@ -5823,9 +5822,7 @@ exptilde(char *startp, char *p, int flags)
        if (!home || !*home)
                goto lose;
        *p = c;
-       startloc = expdest - (char *)stackblock();
        strtodest(home, SQSYNTAX, quotes);
-       recordregion(startloc, expdest - (char *)stackblock(), 0);
        return p;
  lose:
        *p = c;