Re: Two noncritical bugs of pg_waldump - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Two noncritical bugs of pg_waldump
Date
Msg-id 20220225184847.ft7wn6e6diflz5fi@alap3.anarazel.de
Whole thread Raw
In response to Re: Two noncritical bugs of pg_waldump  (Kyotaro Horiguchi <horikyota.ntt@gmail.com>)
Responses Re: Two noncritical bugs of pg_waldump
List pgsql-hackers
Hi,

On 2022-02-14 18:18:47 +0900, Kyotaro Horiguchi wrote:
> > If I give an empty file to the tool it complains as the follows.
> > 
> > > pg_waldump: fatal: could not read file "hoge": No such file or directory
> > 
> > No, the file exists.  The cause is it reads uninitialized errno to
> > detect errors from the system call.  read(2) is defined to set errno
> > always when it returns -1 and doesn't otherwise. Thus it seems to me
> > that it is better to check that the return value is less than zero
> > than to clear errno before the call to read().
> 
> So I post a patch contains only the indisputable part.

Thanks for the report and fix. Pushed. This was surprisingly painful, all but
one branch had conflicts...

Greetings,

Andres Freund



pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Using operators to do query hints
Next
From: David Christensen
Date:
Subject: Re: [PATCH] add relation and block-level filtering to pg_waldump