summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTom Lane2009-02-03 00:59:26 +0000
committerTom Lane2009-02-03 00:59:26 +0000
commit2327a395624f9a8a6592447251f5a90c26918278 (patch)
tree7f1f00bed285f49122bc9dc3066edaacfd8dc1ac
parent0b64c295b6f3dd70a18d9b18e2f58fdea1ff6fa4 (diff)
Add a failure check for syslogger's use of _beginthreadex(), and remove
unnecessary thread address output parameter, to make this code look more like that in pg_restore.
-rw-r--r--src/backend/postmaster/syslogger.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/src/backend/postmaster/syslogger.c b/src/backend/postmaster/syslogger.c
index 461c1ffbe8..988b0e04ff 100644
--- a/src/backend/postmaster/syslogger.c
+++ b/src/backend/postmaster/syslogger.c
@@ -267,11 +267,9 @@ SysLoggerMain(int argc, char *argv[])
/* Fire up separate data transfer thread */
InitializeCriticalSection(&sysfileSection);
- {
- unsigned int tid;
-
- threadHandle = (HANDLE) _beginthreadex(0, 0, pipeThread, 0, 0, &tid);
- }
+ threadHandle = (HANDLE) _beginthreadex(NULL, 0, pipeThread, NULL, 0, NULL);
+ if (threadHandle == 0)
+ elog(FATAL, "could not create syslogger data transfer thread: %m");
#endif /* WIN32 */
/* remember active logfile parameters */