Hi. I just backed out revision 1.21 of imgact_shell.c. As mentioned in commit log, no other operating system has this magic. I'm also going to MFC it to RELENG_5. Under really big claims, we can consider MFCing this fix: http://people.freebsd.org/~pjd/patches/shell2.patch which adds backward compatibility for this "feature", but it's ugly and I'll be glad if we can avoid this. The commit: pjd 2004-10-31 11:12:59 UTC FreeBSD src repository Modified files: sys/kern imgact_shell.c Log: Don't treat # as a comment in interpreter specification line. This is magic and no other operating system do so (i.e. Solaris, Tru64, Linux, AIX, HP-UX, Irix, MacOS X, NetBSD). Discussed on: current_at_ Reported by: Sławek Żak <zaks_at_prioris.mini.pw.edu.pl> Revision Changes Path 1.27 +1 -1 src/sys/kern/imgact_shell.c http://cvsweb.FreeBSD.org/src/sys/kern/imgact_shell.c.diff?r1=1.26&r2=1.27 =================================================================== RCS file: /usr/local/www/cvsroot/FreeBSD/src/sys/kern/imgact_shell.c,v retrieving revision 1.26 retrieving revision 1.27 diff -u -p -r1.26 -r1.27 --- src/sys/kern/imgact_shell.c 2003/06/11 00:56:54 1.26 +++ src/sys/kern/imgact_shell.c 2004/10/31 11:12:59 1.27 _at__at_ -25,7 +25,7 _at__at_ */ #include <sys/cdefs.h> -__FBSDID("$FreeBSD: /usr/local/www/cvsroot/FreeBSD/src/sys/kern/imgact_shell.c,v 1.26 2003/06/11 00:56:54 obrien Exp $"); +__FBSDID("$FreeBSD: /usr/local/www/cvsroot/FreeBSD/src/sys/kern/imgact_shell.c,v 1.27 2004/10/31 11:12:59 pjd Exp $"); #include <sys/param.h> #include <sys/systm.h> _at__at_ -73,7 +73,7 _at__at_ exec_shell_imgact(imgp) /* * Find end of line; return if the line > MAXSHELLCMDLEN long. */ - for (ihp = &image_header[2]; *ihp != '\n' && *ihp != '#'; ++ihp) { + for (ihp = &image_header[2]; *ihp != '\n'; ++ihp) { if (ihp >= &image_header[MAXSHELLCMDLEN]) return(ENAMETOOLONG); } -- Pawel Jakub Dawidek http://www.FreeBSD.org pjd_at_FreeBSD.org http://garage.freebsd.pl FreeBSD committer Am I Evil? Yes, I Am!
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:38:20 UTC