From e23fe25b33324a9ea992276c1a4f04127bf9ba4b Mon Sep 17 00:00:00 2001 From: Andreas Schwab Date: Sun, 20 Feb 2011 07:24:56 -0500 Subject: [PATCH] Move setting variable in relro data earlier in ld.so. --- ChangeLog | 8 ++++++++ NEWS | 2 +- elf/rtld.c | 11 +++++++---- 3 files changed, 16 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 47eb8c39df..b47a0133f5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2010-09-28 Andreas Schwab + Ulrich Drepper + + [BZ #12489] + * elf/rtld.c (dl_main): Move setting of GLRO(dl_init_all_dirs) + before performing relro protection. At old place add assertion + to make sure nothing changed. + 2011-02-17 Nathan Sidwell Glauber de Oliveira Costa diff --git a/NEWS b/NEWS index 8d961b07bb..da091f5fad 100644 --- a/NEWS +++ b/NEWS @@ -9,7 +9,7 @@ Version 2.14 * The following bugs are resolved with this release: - 11724, 12460, 12469 + 11724, 12460, 12469, 12489 Version 2.13 diff --git a/elf/rtld.c b/elf/rtld.c index 851038089c..174954bf3b 100644 --- a/elf/rtld.c +++ b/elf/rtld.c @@ -1,5 +1,5 @@ /* Run time dynamic linker. - Copyright (C) 1995-2006, 2007, 2008, 2009, 2010 Free Software Foundation, Inc. + Copyright (C) 1995-2010, 2011 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -2179,6 +2179,10 @@ ERROR: ld.so: object '%s' cannot be loaded as audit interface: %s; ignored.\n", we need it in the memory handling later. */ GLRO(dl_initial_searchlist) = *GL(dl_ns)[LM_ID_BASE]._ns_main_searchlist; + /* Remember the last search directory added at startup, now that + malloc will no longer be the one from dl-minimal.c. */ + GLRO(dl_init_all_dirs) = GL(dl_all_dirs); + if (prelinked) { if (main_map->l_info [ADDRIDX (DT_GNU_CONFLICT)] != NULL) @@ -2298,9 +2302,8 @@ ERROR: ld.so: object '%s' cannot be loaded as audit interface: %s; ignored.\n", lossage); } - /* Remember the last search directory added at startup, now that - malloc will no longer be the one from dl-minimal.c. */ - GLRO(dl_init_all_dirs) = GL(dl_all_dirs); + /* Make sure no new search directories have been added. */ + assert (GLRO(dl_init_all_dirs) == GL(dl_all_dirs)); if (! prelinked && rtld_multiple_ref) {