Re: [HACKERS] OpenTemporaryFile() vs resowner.c

2017-11-08 Thread Tom Lane
Thomas Munro writes: > Andres, Robert and Peter G rightly complained[1] that my shared > temporary file patch opens a file, then calls > ResourceOwnerEnlargeFiles() which can fail due to lack of memory, and > then registers the file handle to make sure we don't leak

[HACKERS] OpenTemporaryFile() vs resowner.c

2017-11-07 Thread Thomas Munro
Hi hackers, Andres, Robert and Peter G rightly complained[1] that my shared temporary file patch opens a file, then calls ResourceOwnerEnlargeFiles() which can fail due to lack of memory, and then registers the file handle to make sure we don't leak it. Doh. The whole point of the separate