Re: [PATCH] bug in fts_open

2010-01-28 Thread Daniel Dickman
On Thu, 28 Jan 2010, Otto Moerbeek wrote: > On Thu, Jan 28, 2010 at 03:25:21AM -0500, Daniel Dickman wrote: > > > Note: the patch below is NOT recommended for 4.7 as this will change > > the semantics of fts_open and will BREAK things like "gzip -" and > > possibly other stuff. > > > > Calls to

Re: [PATCH] bug in fts_open

2010-01-28 Thread Otto Moerbeek
On Thu, Jan 28, 2010 at 03:25:21AM -0500, Daniel Dickman wrote: > Note: the patch below is NOT recommended for 4.7 as this will change > the semantics of fts_open and will BREAK things like "gzip -" and > possibly other stuff. > > Calls to fts_open should probably be audited first... yes, this i

Re: [PATCH] bug in fts_open

2010-01-28 Thread Daniel Dickman
Note: the patch below is NOT recommended for 4.7 as this will change the semantics of fts_open and will BREAK things like "gzip -" and possibly other stuff. Calls to fts_open should probably be audited first... On Thu, Jan 28, 2010 at 2:06 AM, Daniel Dickman wrote: > As reported on the git mail

[PATCH] bug in fts_open

2010-01-27 Thread Daniel Dickman
As reported on the git mailing list[1], find(1) should exit with non-zero status when a non-existent path is given which I think is also the intent of the find(1) code. [1] http://marc.info/?l=git&m=126186443524638 For example: # find /no-such-path find: fts_open: No such file