Re: pg_archivecleanup bug

On Thu, Mar 13, 2014 at 1:48 AM, Bruce Momjian <bruce(at)momjian(dot)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 <tgl(at)sss(dot)pgh(dot)pa(dot)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