Don't trust CreateFileMapping() to clear the error code on success. We must test GetLastError() even when CreateFileMapping() returns a non-null handle. If that value were left over from some previous system call, we might be fooled into thinking the segment already existed. Experimentation on Windows 7 suggests that CreateFileMapping() clears the error code on success, but it is not documented to do so, so let's not rely on that happening in all Windows releases.
Amit Kapila Discussion: <[email protected]> Branch ------ REL9_4_STABLE Details ------- http://git.postgresql.org/pg/commitdiff/b2d59ab02ce380daec0b974955d047eb74b0cebb Modified Files -------------- src/backend/storage/ipc/dsm_impl.c | 3 +++ 1 file changed, 3 insertions(+) -- Sent via pgsql-committers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-committers
