From owner-cvs-src-old@FreeBSD.ORG Wed Apr 20 22:26:59 2011 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 301DF106566B for ; Wed, 20 Apr 2011 22:26:59 +0000 (UTC) (envelope-from jilles@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 1DF7E8FC08 for ; Wed, 20 Apr 2011 22:26:59 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.4/8.14.4) with ESMTP id p3KMQxGT085953 for ; Wed, 20 Apr 2011 22:26:59 GMT (envelope-from jilles@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.4/8.14.4/Submit) id p3KMQw7i085952 for cvs-src-old@freebsd.org; Wed, 20 Apr 2011 22:26:59 GMT (envelope-from jilles@repoman.freebsd.org) Message-Id: <201104202226.p3KMQw7i085952@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to jilles@repoman.freebsd.org using -f From: Jilles Tjoelker Date: Wed, 20 Apr 2011 22:24:54 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: HEAD Subject: cvs commit: src/bin/sh parser.c src/tools/regression/bin/sh/expansion length6.0 X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Apr 2011 22:26:59 -0000 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)