generic-library/bionic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Return EINVAL when calling pthread_detach for joined thread.

Browse Source 
Change-Id: I717015132187e087e0ad485284a13c8801e25e77
This commit is contained in:
Yabin Cui 2015-03-19 15:19:25 -07:00
parent 7875b506ed
commit bbb0432a33
2 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
libc/bionic/pthread_detach.cpp
View File

@ -44,7 +44,7 @@ int pthread_detach(pthread_t t) {
}
switch (old_state) {
case THREAD_NOT_JOINED: return 0;
case THREAD_JOINED: return 0; // Already being joined; silently do nothing, like glibc.
case THREAD_JOINED: return EINVAL;
case THREAD_DETACHED: return EINVAL;
case THREAD_EXITED_NOT_JOINED: break; // Call pthread_join out of scope of pthread_accessor.
}

5
tests/pthread_test.cpp
View File

@ -271,8 +271,11 @@ TEST(pthread, pthread_no_op_detach_after_join) {
sleep(1); // (Give t2 a chance to call pthread_join.)
// ...a call to pthread_detach on thread 1 will "succeed" (silently fail)...
#if defined(__BIONIC__)
ASSERT_EQ(EINVAL, pthread_detach(t1));
#else
ASSERT_EQ(0, pthread_detach(t1));
#endif
AssertDetached(t1, false);
spinhelper.UnSpin();