Ensure __set_errno is still visible on LP32.

The use of the .hidden directive to avoid going via the PLT for
__set_errno had the side-effect of actually making __set_errno
hidden (which is odd because assembler directives don't usually
affect symbols defined in a different file --- you can't even
create a weak reference to a symbol that's defined in a different
file).

This change switches the system call stubs over to a new always-hidden
__set_errno_internal and has a visible __set_errno on LP32 just for
binary compatibility with old NDK apps.

(cherry-pick of 7efad83d430f4d824f2aaa75edea5106f6ff8aae.)

Bug: 17423135
Change-Id: I6b6d7a05dda85f923d22e5ffd169a91e23499b7b
This commit is contained in:
Elliott Hughes
2014-09-08 15:25:01 -07:00
parent 201f36d23b
commit 011e111d29
1176 changed files with 1193 additions and 3503 deletions

View File

@@ -2,8 +2,6 @@
#include <private/bionic_asm.h>
.hidden __set_errno
ENTRY(writev)
movl $__NR_writev, %eax
syscall
@@ -11,7 +9,7 @@ ENTRY(writev)
jb 1f
negl %eax
movl %eax, %edi
call __set_errno
call __set_errno_internal
1:
ret
END(writev)