Re: [PATCH] switch stat and fopen

2007-02-13 Thread Marcus Meissner
On Tue, Feb 13, 2007 at 05:28:52PM +0900, Mike McCormack wrote: > > Marcus Meissner wrote: > > >-if (!(fin = fopen(input_file_name, "r"))) return 0; > > if (stat(input_file_name, &st) < 0) return 0; > >+if (!(fin = fopen(input_file_name, "r"))) return 0; > > Wouldn't it be better to

Re: [PATCH] switch stat and fopen

2007-02-13 Thread Mike McCormack
Marcus Meissner wrote: -if (!(fin = fopen(input_file_name, "r"))) return 0; if (stat(input_file_name, &st) < 0) return 0; +if (!(fin = fopen(input_file_name, "r"))) return 0; Wouldn't it be better to use fstat here to avoid a race condition? Mike