diff options
author | Mark Wielaard <mjw@redhat.com> | 2015-06-03 18:50:40 +0200 |
---|---|---|
committer | Mark Wielaard <mjw@redhat.com> | 2015-06-09 22:47:54 +0200 |
commit | 9a9f7f6667ae156c8fb8e1290d855d8b8e893461 (patch) | |
tree | 0efd12fe7d845d0612b066a5482da028db0d71e0 | |
parent | b07d1655383e1d28166d349b2e780b24f167ed42 (diff) | |
download | android_external_elfutils-9a9f7f6667ae156c8fb8e1290d855d8b8e893461.tar.gz android_external_elfutils-9a9f7f6667ae156c8fb8e1290d855d8b8e893461.tar.bz2 android_external_elfutils-9a9f7f6667ae156c8fb8e1290d855d8b8e893461.zip |
libelf: Make sure conversion functions work on aligned data for type.
The gelf_xlate conversion functions work on properly aligned ELF data
types. If elf_get data needs to do conversion and ! ALLOW_UNALIGNED
and the rawdata_base isn't aligned properly for the section type, then
provide an aligned copy of the data.
Found with --enable-sanitize-undefined in run-test-archive64.sh on x86_64.
Signed-off-by: Mark Wielaard <mjw@redhat.com>
-rw-r--r-- | libelf/ChangeLog | 5 | ||||
-rw-r--r-- | libelf/elf_getdata.c | 26 |
2 files changed, 29 insertions, 2 deletions
diff --git a/libelf/ChangeLog b/libelf/ChangeLog index 772eb52d..25f673db 100644 --- a/libelf/ChangeLog +++ b/libelf/ChangeLog @@ -1,3 +1,8 @@ +2015-06-02 Mark Wielaard <mjw@redhat.com> + + * elf_getdata.c (convert_data): Make sure source data is properly + aligned for type before calling actual conversion function. + 2015-06-04 Mark Wielaard <mjw@redhat.com> * elf_begin.c (get_shnum): Check alignment of Shdr, not Ehdr before diff --git a/libelf/elf_getdata.c b/libelf/elf_getdata.c index 8567da1e..1a4981ef 100644 --- a/libelf/elf_getdata.c +++ b/libelf/elf_getdata.c @@ -1,5 +1,5 @@ /* Return the next data element from the section after possibly converting it. - Copyright (C) 1998-2005, 2006, 2007 Red Hat, Inc. + Copyright (C) 1998-2005, 2006, 2007, 2015 Red Hat, Inc. This file is part of elfutils. Written by Ulrich Drepper <drepper@redhat.com>, 1998. @@ -144,6 +144,25 @@ convert_data (Elf_Scn *scn, int version __attribute__ ((unused)), int eclass, return; } + /* Make sure the source is correctly aligned for the conversion + function to directly access the data elements. */ + char *rawdata_source; + if (ALLOW_UNALIGNED || + ((((size_t) (char *) scn->rawdata_base)) & (align - 1)) == 0) + rawdata_source = scn->rawdata_base; + else + { + rawdata_source = (char *) malloc (size); + if (rawdata_source == NULL) + { + __libelf_seterrno (ELF_E_NOMEM); + return; + } + + /* The copy will be appropriately aligned for direct access. */ + memcpy (rawdata_source, scn->rawdata_base, size); + } + /* Get the conversion function. */ #if EV_NUM != 2 fp = __elf_xfctstom[version - 1][__libelf_version - 1][eclass - 1][type]; @@ -151,7 +170,10 @@ convert_data (Elf_Scn *scn, int version __attribute__ ((unused)), int eclass, fp = __elf_xfctstom[0][0][eclass - 1][type]; #endif - fp (scn->data_base, scn->rawdata_base, size, 0); + fp (scn->data_base, rawdata_source, size, 0); + + if (rawdata_source != scn->rawdata_base) + free (rawdata_source); } scn->data_list.data.d.d_buf = scn->data_base; |