Into my FreeBSD fix I've also included '\0' into the list of possible
line terminators. Previously '\0' has been considered as "ordinary"
character, neither token separator nor line separator, which is probably
incorrect. Also I think your patch is partially incorrect, since it
ignores situation when interpreter string exceeds MAXSHELLCMDLEN - it
just truncates name in this case.
-Maxim
Matthew Dillon wrote:
> dillon 2005/02/25 00:49:10 PST
>
> DragonFly src repository
>
> Modified files:
> sys/kern imgact_shell.c
> Log:
> Rewrite the loops which extract the interpreter name and arguments out
> of the #! line for a shell script. The loop was improperly handling a
> missing newline by scanning past the end of the buffer, resulting in a panic.
>
> Reported-by: Garance A Drosihn <drosih@rpi.edu>
>
> Revision Changes Path
> 1.5 +27 -20 src/sys/kern/imgact_shell.c
>
>
> http://www.dragonflybsd.org/cvsweb/src/sys/kern/imgact_shell.c.diff?r1=1.4&r2=1.5&f=u