summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2013-07-22 14:08:33 -0400
committerRich Felker <dalias@aerifal.cx>2013-07-22 14:08:33 -0400
commit1da53dad278f98b7712ac002162afaa8815ba580 (patch)
tree31629639333932714d99aa6d566d8dc2d73d540f
parent4e3c6b430382b152e783454a31f9a884223ce77f (diff)
downloadmusl-1da53dad278f98b7712ac002162afaa8815ba580.tar.gz
disable legacy init/fini processing on ARM
since the old, poorly-thought-out musl approach to init/fini arrays on ARM (when it was the only arch that needed them) was to put the code in crti/crtn and have the legacy _init/_fini code run the arrays, adding proper init/fini array support caused the arrays to get processed twice on ARM. I'm not sure skipping legacy init/fini processing is the best solution to the problem, but it works, and it shouldn't break anything since the legacy init/fini system was never used for ARM EABI.
-rw-r--r--arch/arm/reloc.h2
-rw-r--r--src/ldso/dynlink.c4
2 files changed, 6 insertions, 0 deletions
diff --git a/arch/arm/reloc.h b/arch/arm/reloc.h
index 9ca0b48c..264b7ab2 100644
--- a/arch/arm/reloc.h
+++ b/arch/arm/reloc.h
@@ -52,3 +52,5 @@ static inline void do_single_reloc(
break;
}
}
+
+#define NO_LEGACY_INITFINI
diff --git a/src/ldso/dynlink.c b/src/ldso/dynlink.c
index 3a0bf95d..b5e4b72f 100644
--- a/src/ldso/dynlink.c
+++ b/src/ldso/dynlink.c
@@ -702,8 +702,10 @@ static void do_fini()
size_t *fn = (size_t *)(p->base + dyn[DT_FINI_ARRAY])+n;
while (n--) ((void (*)(void))*--fn)();
}
+#ifndef NO_LEGACY_INITFINI
if (dyn[0] & (1<<DT_FINI))
((void (*)(void))(p->base + dyn[DT_FINI]))();
+#endif
}
}
@@ -723,8 +725,10 @@ static void do_init_fini(struct dso *p)
p->fini_next = fini_head;
fini_head = p;
}
+#ifndef NO_LEGACY_INITFINI
if (dyn[0] & (1<<DT_INIT))
((void (*)(void))(p->base + dyn[DT_INIT]))();
+#endif
if (dyn[0] & (1<<DT_INIT_ARRAY)) {
size_t n = dyn[DT_INIT_ARRAYSZ]/sizeof(size_t);
size_t *fn = (void *)(p->base + dyn[DT_INIT_ARRAY]);