commit aec2bb5ec67637ec4b5374287e9bc0d572632054
author Yabin Cui <yabinc@google.com> Thu Mar 19 16:28:19 2015 -0700
committer Yabin Cui <yabinc@google.com> Thu Mar 19 23:35:23 2015 +0000
tree 81a349d5c4f2a4962201fa9aea395014bdc89867
parent 2e49508187254903943b83df7bb241d5fb6a8593

Fix two errors in pthread_detach.cpp.

The errors are introduced in "Make pthread join_state not protected by g_thread_list_lock".

Bug: 19636317
Change-Id: I58ae9711da94bfbac809abfd81311eeb70301a4b

libc/bionic/pthread_detach.cpp

diff --git a/libc/bionic/pthread_detach.cpp b/libc/bionic/pthread_detach.cpp
index 7ae5eb4..9f957f4 100644
--- a/libc/bionic/pthread_detach.cpp
+++ b/libc/bionic/pthread_detach.cpp
@@ -45,7 +45,7 @@
     switch (old_state) {
       case THREAD_NOT_JOINED: return 0;
       case THREAD_JOINED:     return 0;  // Already being joined; silently do nothing, like glibc.
-      case THREAD_DETACHED:   return THREAD_DETACHED;
+      case THREAD_DETACHED:   return EINVAL;
       case THREAD_EXITED_NOT_JOINED: break;  // Call pthread_join out of scope of pthread_accessor.
     }
   }