summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCupertino Miranda <Cupertino.Miranda@synopsys.com>2017-04-06 11:57:25 +0200
committerWaldemar Brodkorb <wbx@openadk.org>2017-04-08 21:44:35 +0200
commit60d5a1ad24f1b853f55dfaf57b23f1588fda1bf7 (patch)
treecc335c1c7f8d7ffda27354964a575dbf42dc80ea
parenta3805b91ecdb458cd20245c20e49deacc58e9c32 (diff)
Same iteration variable used for inner and outer loop
Inner loop was using same counter variable (i) as the outer loop, therefore making outer loop terminate before it visited all of the ELF program segments. Surrounding code in this inner loop clearly shows the intention that this loop should not affect the outer one, therefore leading me to the conclusion that this should be a bug an not expected code. This bug was detected due to some other bug in ARC binutils that kept setting TEXTREL for any PIE application. Apart from the but, I have also moved the debug message inside of the TEXTREL condition as mprotect is only really called if TELTREL is set.
-rw-r--r--ldso/ldso/ldso.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/ldso/ldso/ldso.c b/ldso/ldso/ldso.c
index 539975bcf..c18d96bdb 100644
--- a/ldso/ldso/ldso.c
+++ b/ldso/ldso/ldso.c
@@ -670,12 +670,13 @@ of this helper program; chances are you did not intend to run this program.\n\
* dynamic linking. We can set the protection back
* again once we are done.
*/
- _dl_debug_early("calling mprotect on the application program\n");
/* Now cover the application program. */
if (app_tpnt->dynamic_info[DT_TEXTREL]) {
+ int j;
ElfW(Phdr) *ppnt_outer = ppnt;
+ _dl_debug_early("calling mprotect on the application program\n");
ppnt = (ElfW(Phdr) *) auxvt[AT_PHDR].a_un.a_val;
- for (i = 0; i < auxvt[AT_PHNUM].a_un.a_val; i++, ppnt++) {
+ for (j = 0; j < auxvt[AT_PHNUM].a_un.a_val; j++, ppnt++) {
if (ppnt->p_type == PT_LOAD && !(ppnt->p_flags & PF_W))
_dl_mprotect((void *) (DL_RELOC_ADDR(app_tpnt->loadaddr, ppnt->p_vaddr) & PAGE_ALIGN),
(DL_RELOC_ADDR(app_tpnt->loadaddr, ppnt->p_vaddr) & ADDR_ALIGN) +