Re: [PATCH 0/2] open() error checking

2013-07-16 Thread Thomas Rast
Junio C Hamano writes: > Thomas Rast writes: > >> I originally had a four-patch series to open 0/1/2 from /dev/null, but >> then I noticed that this was shot down in 2008: >> >> http://thread.gmane.org/gmane.comp.version-control.git/93605/focus=93896 > > The way I recall the thread was not "sh

Re: [PATCH 0/2] open() error checking

2013-07-12 Thread Junio C Hamano
Thomas Rast writes: > #1 is Dale's suggested change. Dale, to include it we'd need your > Signed-off-by as per Documentation/SubmittingPatches. > > #2 is a similar error-checking fix; I reviewed 'git grep "\bopen\b"' > and found one case where the return value was obviously not tested. > The cor