On Sun, Dec 05, 2010 at 05:31:59PM +0000, Jilles Tjoelker wrote:
> A tilde expansion generates a valid pathname. Splitting it using IFS
> either leaves it unchanged or changes it to something unintended.
> 
> This fixes FreeBSD sh test expansion/tilde1.0 and does not change the
> outcome of the other tests.
> 
> This fixes Debian bug #601096.
> 
> Example:
>   IFS=m HOME=/tmp; printf "%s\n" ~

Thanks for the patch! I also removed startloc which is now unused.

commit 834629283f6c629a4da05ef60bae9445c954a19a
Author: Jilles Tjoelker <jil...@stack.nl>
Date:   Thu Mar 10 17:16:13 2011 +0800

    [EXPAND] Do not split the result of tilde expansion
    
    A tilde expansion generates a valid pathname. Splitting it using IFS
    either leaves it unchanged or changes it to something unintended.
    
    This fixes FreeBSD sh test expansion/tilde1.0 and does not change the
    outcome of the other tests.
    
    This fixes Debian bug #601096.
    
    Example:
      IFS=m HOME=/tmp; printf "%s\n" ~
    
    Signed-off-by: Herbert Xu <herb...@gondor.apana.org.au>

diff --git a/ChangeLog b/ChangeLog
index c94dff8..a64e030 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,7 @@
 2011-03-10  Jilles Tjoelker <jil...@stack.nl>
 
        * Replace GPL noclobberopen code with the FreeBSD version.
+       * Do not split the result of tilde expansion.
 
 2010-11-28  Maciej ┼╗enczykowski <zenczykow...@gmail.com>
 
diff --git a/src/expand.c b/src/expand.c
index 1b77b7c..6bee5c5 100644
--- a/src/expand.c
+++ b/src/expand.c
@@ -364,7 +364,6 @@ exptilde(char *startp, char *p, int flag)
        char *name;
        const char *home;
        int quotes = flag & QUOTES_ESC;
-       int startloc;
 
        name = p + 1;
 
@@ -393,9 +392,7 @@ done:
        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;

Cheers,
-- 
Email: Herbert Xu <herb...@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to