Re: errno clobbering in reorderbuffer

From: Andres Freund <andres(at)anarazel(dot)de>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: errno clobbering in reorderbuffer
Date: 2016-08-18 22:18:52
Message-ID: 20160818221852.6tvi5wxsymtuecjz@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2016-08-18 19:06:02 -0300, Alvaro Herrera wrote:
> if (write(fd, rb->outbuf, ondisk->size) != ondisk->size)
> {
> + int save_errno = errno;
> +
> CloseTransientFile(fd);
> + errno = save_errno;
> ereport(ERROR,
> (errcode_for_file_access(),
> errmsg("could not write to data file for XID %u: %m",

Independent of this specific case I kinda wish we had a better way to
deal with exactly this pattern. I even wonder whether having a close
variant not clobbering errno might be worthwhile.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Corey Huinker 2016-08-18 23:02:23 Re: anyelement -> anyrange
Previous Message Tom Lane 2016-08-18 22:17:25 Re: errno clobbering in reorderbuffer