Check the return code of pthread_create(). Otherwise we go into an infinite

loop if it fails, which is what what happened on my HP-UX box. (I think
the reason it failed on that box is a misconfiguration on my behalf, but
that's no reason to hang.)

diff --git a/src/test/thread/thread_test.c b/src/test/thread/thread_test.c
index 0782e8c66bb7b5fd4836ca4e553ab2c56cb139fe..6a8182991295274e1b353e8dc4d9340cd8e9d86a 100644 (file)
--- a/src/test/thread/thread_test.c
+++ b/src/test/thread/thread_test.c
@@ -150,6 +150,7 @@ main(int argc, char *argv[])
        pthread_t       thread1,
                                thread2;
        int                     fd;
+       int                     rc;
 
 #ifdef WIN32
        WSADATA         wsaData;
@@ -199,8 +200,23 @@ main(int argc, char *argv[])
        /* Hold lock until we are ready for the child threads to exit. */
        pthread_mutex_lock(&init_mutex);
 
-       pthread_create(&thread1, NULL, (void *(*) (void *)) func_call_1, NULL);
-       pthread_create(&thread2, NULL, (void *(*) (void *)) func_call_2, NULL);
+       rc = pthread_create(&thread1, NULL, (void *(*) (void *)) func_call_1, NULL);
+       if (rc != 0)
+       {
+               fprintf(stderr, "Failed to create thread 1: %s **\nexiting\n",
+                               strerror(rc));
+               exit(1);
+       }
+       rc = pthread_create(&thread2, NULL, (void *(*) (void *)) func_call_2, NULL);
+       if (rc != 0)
+       {
+               /*
+                * strerror() might not be thread-safe, and we already spawned thread
+                * 1 that uses it
+                */
+               fprintf(stderr, "Failed to create thread 2 **\nexiting\n");
+               exit(1);
+       }
 
        while (thread1_done == 0 || thread2_done == 0)
                sched_yield();                  /* if this is a portability problem, remove it */