On Thu, Mar 13, 2014 at 1:48 AM, Bruce Momjian <br...@momjian.us> wrote: > On Mon, Dec 9, 2013 at 11:27:28AM -0500, Robert Haas wrote: >> On Thu, Dec 5, 2013 at 6:15 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: >> > But the other usages seem to be in assorted utilities, which >> > will need to do it right for themselves. initdb.c's walkdir() seems to >> > have it right and might be a reasonable model to follow. Or maybe we >> > should invent a frontend-friendly version of ReadDir() rather than >> > duplicating all the error checking code in ten-and-counting places? >> >> If there's enough uniformity in all of those places to make that >> feasible, it certainly seems wise to do it that way. I don't know if >> that's the case, though - e.g. maybe some callers want to exit and >> others do not. pg_resetxlog wants to exit; pg_archivecleanup and >> pg_standby most likely want to print an error and carry on. > > I have developed the attached patch which fixes all cases where > readdir() wasn't checking for errno, and cleaned up the syntax in other > cases to be consistent.
Thanks! > While I am not a fan of backpatching, the fact we are ignoring errors in > some critical cases seems the non-cosmetic parts should be backpatched. While I haven't read the patch, I agree that this is a back-patchable bug fix. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
