summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* improve readability of stdio: fix indentation and remove trailing spacesAndré Goddard Rosa2010-05-1715-57/+56
| | | | | Change-Id: Ic51e58a7c75d20bf770dc0ebd7f97a338fbe0036 Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* Merge "Add declaration for sys_signame[] forgotten in change 12489"David Turner2010-05-141-0/+1
|\
| * Add declaration for sys_signame[] forgotten in change 12489Thorsten Glaser2010-04-051-0/+1
| |
* | Set SA_RESTORER in sigaction()Matt Fischer2010-05-128-6/+50
| | | | | | | | | | | | | | | | | | GDB looks for specific opcode sequences when trying to recognize a stack frame as a signal trampoline. The sequences it looks for happen to be those created when SA_RESTORER is set, since glibc always sets a restorer. This patch does the same here, so that the trampolines can be correctly identified. Change-Id: I0ac574a68818cb24d939c3527f3aaeb04b853d04
* | Merge "Sometimes the compiler is very right! Found bug in bessel routines ↵Jean-Baptiste Queru2010-05-113-5/+7
|\ \ | | | | | | | | | for float."
| * | Sometimes the compiler is very right! Found bug in bessel routines for float.Scott Turner2010-05-073-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original compiler error: target arm C: libm <= bionic/libm/src/e_j0f.c bionic/libm/src/e_j0f.c: In function 'j0f': bionic/libm/src/e_j0f.c:66: warning: comparison between signed and unsigned integer expressions bionic/libm/src/e_j0f.c: In function 'y0f': bionic/libm/src/e_j0f.c:140: warning: comparison between signed and unsigned integer expressions target arm C: libm <= bionic/libm/src/e_j1.c It's subtle but ix is masked with 0x7f000000 so it can never ever have a value greater than 0x80000000. So I switched to using the unmasked hx and added a cast as a reward to the compiler for being right. I checked the original routines that e_j0f.c was ported from (in e_j0.c) and the double's don't use 0x80000000 so this issue didn't exist there. Let that be a warning to those that just slap on casts to shut up the compiler, sometimes it's sniffed out a bug for you. :-) Similar fixes in the other functions. Change-Id: I7a776e5d4721fc3a9e3bd89179b67e9af3a2ebfa
* | | Merge "Reduce compiler anxiety."David Turner2010-05-111-1/+1
|\ \ \
| * | | Reduce compiler anxiety.Scott Turner2010-01-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Warning from compiler: target arm C: libm <= bionic/libm/src/e_atan2.c bionic/libm/src/e_atan2.c: In function 'atan2': bionic/libm/src/e_atan2.c:71: warning: suggest parentheses around arithmetic in operand of '|' target arm C: libm <= bionic/libm/src/e_atan2f.c
* | | | Merge "Added support for LD_PRELOAD"David Turner2010-05-111-0/+65
|\ \ \ \
| * | | | Added support for LD_PRELOADMatt Fischer2010-05-101-0/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The LD_PRELOAD environment variable allows the user to specify a list of libraries which should be unconditionally loaded before any others. This makes possible some useful tricks, such as library interposers. Change-Id: I433d775ab08ef63a5fbe7b21f87a5642954fc32f
* | | | | Merge "stdio: simplify __fremovelock()"David Turner2010-05-101-3/+1
|\ \ \ \ \
| * | | | | stdio: simplify __fremovelock()André Goddard Rosa2010-05-101-3/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... by removing extraneous NULL check, as free() already does it. Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com> Change-Id: I0445f35c7ad0a049a0e4aee1fbe002ed2f13b94b
* | | | | Merge "Reversed order of const and static to hush warning from compiler."Jean-Baptiste Queru2010-05-101-10/+10
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Reversed order of const and static to hush warning from compiler.Scott Turner2010-01-161-10/+10
| |/ / /
* | | | Merge "string: tidy up strndup()"David Turner2010-05-101-4/+6
|\ \ \ \ | |_|/ / |/| | |
| * | | string: tidy up strndup()André Goddard Rosa2010-01-301-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It decreases code size: text data bss dec hex filename 161 0 0 161 a1 strndup-BEFORE.o 153 0 0 153 99 strndup-AFTER.o Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* | | | Merge "Fix log channel initialization at bionic/logd_write.c."David Turner2010-05-011-0/+2
|\ \ \ \
| * | | | Fix log channel initialization at bionic/logd_write.c.Alexey Tarasov2009-12-031-0/+2
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | log_channel_t contains fd member, which is file descriptor for exact logging channel. change cc05d1271680d6a7804bc89d3b1fe14c40b32396 lacks initialization of this member in __write_to_log_init(), thus logging code is not working, although not crashable. Additional details may be found in comments here: http://review.source.android.com/5617
* | | | Merge "added headers for SuperH which automatically generate by update_all.py"Jean-Baptiste Queru2010-04-29174-0/+7188
|\ \ \ \
| * | | | added headers for SuperH which automatically generate by update_all.pyTony SIM2010-02-01174-0/+7188
| |/ / / | | | | | | | | | | | | base on kernel 2.6.27 arch/sh/include/asm/
* | | | linker: fix the undefined weak symbols issueYi Sun2010-04-290-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | I'm not sure if this is a correct fix or not. Also need to find out why 1.6 does not have this issue. Change-Id: If46c844834bda1e2cbf084a1a45a1832119b3ae3
* | | | bionic: add missing NULL check from memory allocation on record_backtrace()André Goddard Rosa2010-04-291-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com> Change-Id: I739c63c5a7344fff1775580044dc647edf246ebf
* | | | stdio: simplify vasprintf()André Goddard Rosa2010-04-291-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... by removing unneeded NULL check, as free() already does it. By the way, we don't need to set a stack variable back to NULL. Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com> Change-Id: Id90eb8f042b5c922c5ff139b11ff8366fb404566
* | | | stdio: simplify asprintf()André Goddard Rosa2010-04-291-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... by removing unneeded NULL check, as free() already does it. By the way, we don't need to set a stack variable back to NULL. Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com> Change-Id: Id1f72e872f73366dddcea4abc75885a3d9a318c6
* | | | improve readability of stdlib: fix indentation and remove trailing spacesAndré Goddard Rosa2010-04-2911-232/+230
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com> Change-Id: I7dd90a0816b5376ffc1de4499d56935e0bd574a1
* | | | Fixed support for RTLD_NEXT in dlsym()Matt Fischer2010-04-083-5/+15
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | The previous implementation of this flag was broken--it behaved identically to RTLD_DEFAULT. This adds a proper implementation, which examines the address of the calling function, and uses it to determine which library to use to begin the symbol search process. Change-Id: I2ad2b46363f68932af63a3828a22f9c7987eea67
* | | bionic: fix memory leak in get_malloc_leak_info() error pathAndré Goddard Rosa2010-03-291-1/+2
| | | | | | | | | | | | Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* | | Merge "bionic: clear only the part of the buffer which is not overwritten ↵David Turner2010-03-301-2/+1
|\ \ \ | | | | | | | | | | | | afterward"
| * | | bionic: clear only the part of the buffer which is not overwritten afterwardAndré Goddard Rosa2010-03-281-2/+1
| | | | | | | | | | | | | | | | | | | | Change-Id: I5ddd93f0557e5a7401460dc9fc8a55b330a79c3a Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* | | | stdlib: optimize bsearch()André Goddard Rosa2010-03-281-2/+2
|/ / / | | | | | | | | | | | | | | | | | | ... by checking most probable condition first (elements do differ) Change-Id: I424eab9c32a6d9eb82b686ca04025ec8c9097035 Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* | | Merge "linker: fix the undefined weak symbols issue"David Turner2010-03-291-1/+1
|\ \ \
| * | | linker: fix the undefined weak symbols issueMin-su, Kim2010-03-291-1/+1
| | | |
* | | | Merge changes Ibdc6e3c8,I9bcb91a2David Turner2010-03-292-5/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Correct generic memset implementation Generic memcpy should define MEMCOPY before including bcopy.c
| * | | | Correct generic memset implementationChris Dearman2010-02-051-4/+4
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Chris Dearman <chris@mips.com>
| * | | | Generic memcpy should define MEMCOPY before including bcopy.cChris Dearman2010-02-051-1/+1
| | |/ / | |/| | | | | | | | | | Signed-off-by: Chris Dearman <chris@mips.com>
* | | | Merge "Atom optimized string and memory routines"David Turner2010-03-2914-8/+7360
|\ \ \ \ | |_|/ / |/| | |
| * | | Atom optimized string and memory routinesBruce Beare2010-03-2614-8/+7360
| | | | | | | | | | | | | | | | Change-Id: I27b68bb28551c75c9ac84bb9730e2cd8254d8991
* | | | Android hack for size_tBruce Beare2010-03-261-3/+16
|/ / / | | | | | | | | | Change-Id: I3c967fca60f542459dc17be84da47b3b6a26242b
* | | Merge "bionic: on pthread_join(), avoid extra check in case we find the thread"David Turner2010-03-181-5/+4
|\ \ \
| * | | bionic: on pthread_join(), avoid extra check in case we find the threadAndré Goddard Rosa2010-02-051-5/+4
| |/ / | | | | | | | | | | | | | | | ... by using similar logic as used in pthread_detach(). Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* | | Merge "bionic: ftell() returns a long, not an int"David Turner2010-03-181-1/+1
|\ \ \
| * | | bionic: ftell() returns a long, not an intAndré Goddard Rosa2010-02-051-1/+1
| |/ / | | | | | | | | | Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* | | Added support for dladdr()Matt Fischer2010-03-175-15/+101
| | | | | | | | | | | | | | | | | | | | | | | | dladdr() is a GNU extension function, which allows the caller to retrieve symbol information for a specified memory address. It is useful for things like generating backtrace information at runtime. Change-Id: I3a1def1a6c9c666d93e1e97b7d260dfa5b9b79a9
* | | Merge "x86 syscall system call implementation"David Turner2010-03-172-0/+53
|\ \ \
| * | | x86 syscall system call implementationBruce Beare2010-03-042-0/+53
| | | |
* | | | Merge "Fix pread()/pwrite() stubs"David Turner2010-03-172-4/+4
|\ \ \ \
| * | | | Fix pread()/pwrite() stubsMatt Fischer2010-02-122-4/+4
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | On ARM EABI, 64-bit function parameters must be aligned to an even/odd register pair. The weird way these stubs were written (using separate lo/hi parameters) prevented this alignment from being enforced by the compiler.
* | | | Merge "bionic: remove unneeded variable from chk_realloc()"David Turner2010-03-161-2/+3
|\ \ \ \
| * | | | bionic: remove unneeded variable from chk_realloc()André Goddard Rosa2010-02-051-2/+3
| |/ / / | | | | | | | | | | | | | | | | | | | | ... and simplify the generated code. Signed-off-by: André Goddard Rosa <andre.goddard@gmail.com>
* | | | Merge "bionic: equalize the <unknown> program name between ssp.c and ↵David Turner2010-03-161-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | libc_init_common.c"