]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Ensure _dosmaperr() actually sets errno correctly.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 23 Nov 2013 23:24:26 +0000 (18:24 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 23 Nov 2013 23:24:50 +0000 (18:24 -0500)
commitb2502ecef92c598f6d71d80b8a5745d252cfdde5
tree6681a5c7096ef0af8470de37a00ccf5b4465cf0d
parente1f7173ea19ad8977311dc72fe50492fae5c078d
Ensure _dosmaperr() actually sets errno correctly.

If logging is enabled, either ereport() or fprintf() might stomp on errno
internally, causing this function to return the wrong result.  That might
only end in a misleading error report, but in any code that's examining
errno to decide what to do next, the consequences could be far graver.

This has been broken since the very first version of this file in 2006
... it's a bit astonishing that we didn't identify this long ago.

Reported by Amit Kapila, though this isn't his proposed fix.
src/port/win32error.c