Date: Wed, 20 Apr 2011 22:24:54 +0000 (UTC) From: Jilles Tjoelker <jilles@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/bin/sh parser.c src/tools/regression/bin/sh/expansion length6.0 Message-ID: <201104202226.p3KMQw7i085952@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
jilles 2011-04-20 22:24:54 UTC
FreeBSD src repository
Modified files:
bin/sh parser.c
Added files:
tools/regression/bin/sh/expansion length6.0
Log:
SVN rev 220903 on 2011-04-20 22:24:54Z by jilles
sh: Do not word split "${#parameter}".
This is only a problem if IFS contains digits, which is unusual but valid.
Because of an incorrect fix for PR bin/12137, "${#parameter}" was treated
as ${#parameter}. The underlying problem was that "${#parameter}"
erroneously added CTLESC bytes before determining the length. This
was properly fixed for PR bin/56147 but the incorrect fix was not backed
out.
Reported by: Seeker on forums.freebsd.org
MFC after: 2 weeks
Revision Changes Path
1.107 +2 -2 src/bin/sh/parser.c
1.1 +8 -0 src/tools/regression/bin/sh/expansion/length6.0 (new)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201104202226.p3KMQw7i085952>