aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Metcalf <cmetcalf@tilera.com>2013-07-22 11:46:44 -0400
committerChris Metcalf <cmetcalf@tilera.com>2013-07-23 07:47:59 -0400
commit15256e58adc62d8d62c03b2b3ba526b468758702 (patch)
tree09d876faac2b7960eec60da8fe927325b199f14f
parentca4023620a265f90a4022b98ac481fd1d053723a (diff)
downloadglibc-15256e58adc62d8d62c03b2b3ba526b468758702.zip
glibc-15256e58adc62d8d62c03b2b3ba526b468758702.tar.gz
glibc-15256e58adc62d8d62c03b2b3ba526b468758702.tar.bz2
tile BZ #15759: Fix bug in _dl_unmap
We returned without calling __munmap if not in the simulator. Now we call a separate sim_dlclose() function to make the control flow work correctly. (cherry picked from commit 1fe2988f523ddbad93ca7abc98fea982f2ae0505) Conflicts: NEWS
-rw-r--r--NEWS2
-rw-r--r--ports/ChangeLog.tile6
-rw-r--r--ports/sysdeps/tile/dl-runtime.c12
3 files changed, 16 insertions, 4 deletions
diff --git a/NEWS b/NEWS
index da0f312..10c1ca6 100644
--- a/NEWS
+++ b/NEWS
@@ -8,7 +8,7 @@ using `glibc' in the "product" field.
Version 2.17.1
* The following bugs are resolved with this release:
- 15003, 15006, 15122.
+ 15003, 15006, 15122, 15759.
Version 2.17
diff --git a/ports/ChangeLog.tile b/ports/ChangeLog.tile
index 303311d..f0cd08d 100644
--- a/ports/ChangeLog.tile
+++ b/ports/ChangeLog.tile
@@ -1,3 +1,9 @@
+2013-07-22 Chris Metcalf <cmetcalf@tilera.com>
+
+ [BZ #15759]
+ * sysdeps/tile/dl-runtime.c (sim_dlclose): New function.
+ (_dl_unmap): Call sim_dlclose().
+
2013-06-12 Chris Metcalf <cmetcalf@tilera.com>
* sysdeps/tile/bits/endian.h (__BYTE_ORDER): Default to little.
diff --git a/ports/sysdeps/tile/dl-runtime.c b/ports/sysdeps/tile/dl-runtime.c
index e965828..f697d57 100644
--- a/ports/sysdeps/tile/dl-runtime.c
+++ b/ports/sysdeps/tile/dl-runtime.c
@@ -127,8 +127,8 @@ _dl_after_load (struct link_map *l)
}
/* Support notifying the simulator about removed objects prior to munmap(). */
-void internal_function
-_dl_unmap (struct link_map *l)
+static void
+sim_dlclose (ElfW(Addr) map_start)
{
int shift;
@@ -144,9 +144,15 @@ _dl_unmap (struct link_map *l)
DLPUTC ('0');
DLPUTC ('x');
for (shift = (int) sizeof (unsigned long) * 8 - 4; shift >= 0; shift -= 4)
- DLPUTC ("0123456789abcdef"[(l->l_map_start >> shift) & 0xF]);
+ DLPUTC ("0123456789abcdef"[(map_start >> shift) & 0xF]);
DLPUTC ('\0');
+
#undef DLPUTC
+}
+void internal_function
+_dl_unmap (struct link_map *l)
+{
+ sim_dlclose (l->l_map_start);
__munmap ((void *) l->l_map_start, l->l_map_end - l->l_map_start);
}