On Thu, Apr 16, 2020 at 12:08:09PM +0900, Kyotaro Horiguchi wrote: > I'm surprised to find an old thread about the same issue. > > https://www.postgresql.org/message-id/20160307.174354.251049100.horiguchi.kyotaro%40lab.ntt.co.jp > > But I don't think it's not acceptable that use fake errno for gzclose, > but cfclose properly passes-through the error code from gzclose, so it > is enought that the caller should recognize the difference. A problem with this patch is that we may forget again to add this special error handling if more code paths use cfclose(). As of HEAD, there are three code paths where cfclose() is called but it does not generate an error: two when ending a blob and one when ending a data file. Perhaps it would make sense to just move all this error within the routine itself? Note that it would also mean registering file names in lclContext or equivalent as that's an important piece of the error message. -- Michael
Attachment:
signature.asc
Description: PGP signature
