Date: Sun, 24 Oct 2010 20:45:13 +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/parser func1.0 Message-ID: <201010242045.o9OKjlx4015471@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
jilles 2010-10-24 20:45:13 UTC
FreeBSD src repository
Modified files:
bin/sh parser.c
Added files:
tools/regression/bin/sh/parser func1.0
Log:
SVN rev 214291 on 2010-10-24 20:45:13Z by jilles
sh: Make sure defined functions can actually be called.
Add some conservative checks on function names:
- Disallow expansions or quoting characters; these can only be called via
strange control characters
- Disallow '/'; these functions cannot be called anyway, as exec.c assumes
they are pathnames
- Make the CTL* bytes work properly in function names.
These are syntax errors.
POSIX does not require us to support more than names (letters, digits and
underscores, not starting with a digit), but I do not want to restrict it
that much at this time.
Exp-run done by: pav (with some other sh(1) changes)
Revision Changes Path
1.90 +7 -3 src/bin/sh/parser.c
1.1 +25 -0 src/tools/regression/bin/sh/parser/func1.0 (new)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201010242045.o9OKjlx4015471>