On Tuesday 13 November 2007 09:37, Fernando Silveira wrote: > On Nov 13, 2007 2:01 PM, Cristian Ionescu-Idbohrn > > <[EMAIL PROTECTED]> wrote: > > On Tue, 13 Nov 2007, Cristian Ionescu-Idbohrn wrote: > > > BusyBox v1.8.1 (2007-11-12 19:47:38 CET) multi-call binary > > > > > > /var/empty/ is a directory. > > > > > > # tr '\0' ' ' < /var/empty/ > > > Segmentation fault > > > > Looks like tr does not handle this case: > > > > read(0, 0xd5228, 256) = -1 EISDIR (Is a directory) > > --- SIGSEGV (Segmentation fault) @ 0 (0) --- > > +++ killed by SIGSEGV +++ > > Yep, but the patch follows inline. The read(2) return value was being > stored in an unsigned variable, avoiding the error detection. I also > added an error message, which wasn't implemented.
Applied to svn, thanks! -- vda _______________________________________________ busybox mailing list [email protected] http://busybox.net/cgi-bin/mailman/listinfo/busybox
