cvs commit: src/sys/kern imgact_shell.c

Maxim Sobolev sobomax at FreeBSD.org
Fri Feb 25 02:18:24 PST 2005


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 at xxxxxxx>
  
  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





More information about the Commits mailing list