glx-multithread-clearbuffer: Check pthread_join return value.

Fix Coverity unchecked return value defect.

CID: 1415101
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>

1 files changed, 6 insertions, 3 deletions

diff --git a/tests/glx/glx-multithread-clearbuffer.c b/tests/glx/glx-multithread-clearbuffer.c
index 577fde842..c8b18099e 100644
--- a/tests/glx/glx-multithread-clearbuffer.c
+++ b/tests/glx/glx-multithread-clearbuffer.c
@@ -82,6 +82,7 @@ main(int argc, char **argv)
 {
 	/* Need at least 16 contexts to congest the thread queue. */
 	pthread_t thread[16];
+	bool pass = true;
 
 	XInitThreads();
 
@@ -90,11 +91,13 @@ main(int argc, char **argv)
 	for (int i = 0; i < ARRAY_SIZE(thread); i++)
 		pthread_create(&thread[i], NULL, thread_func, NULL);
 
-	for (int i = 0; i < ARRAY_SIZE(thread); i++)
-		pthread_join(thread[i], NULL);
+	for (int i = 0; i < ARRAY_SIZE(thread); i++) {
+		if (pthread_join(thread[i], NULL) != 0)
+			pass = false;
+	}
 
 	pthread_mutex_destroy(&mutex);
 
-	piglit_report_result(PIGLIT_PASS);
+	piglit_report_result(pass ? PIGLIT_PASS : PIGLIT_FAIL);
 	return 0;
 }