aboutsummaryrefslogtreecommitdiff
path: root/intl
diff options
context:
space:
mode:
authorAdhemerval Zanella Netto <adhemerval.zanella@linaro.org>2022-12-27 18:11:42 -0300
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>2023-03-27 13:57:55 -0300
commit88677348b4de73874ca7d5a47451f42880f65f07 (patch)
tree9ebe22b88c2487331521737dfdf089f9e3e2439f /intl
parente4d336f1ace7c7ca535f7f85485373752bc76ed5 (diff)
downloadglibc-88677348b4de73874ca7d5a47451f42880f65f07.zip
glibc-88677348b4de73874ca7d5a47451f42880f65f07.tar.gz
glibc-88677348b4de73874ca7d5a47451f42880f65f07.tar.bz2
Move libc_freeres_ptrs and libc_subfreeres to hidden/weak functions
They are both used by __libc_freeres to free all library malloc allocated resources to help tooling like mtrace or valgrind with memory leak tracking. The current scheme uses assembly markers and linker script entries to consolidate the free routine function pointers in the RELRO segment and to be freed buffers in BSS. This patch changes it to use specific free functions for libc_freeres_ptrs buffers and call the function pointer array directly with call_function_static_weak. It allows the removal of both the internal macros and the linker script sections. Checked on x86_64-linux-gnu, i686-linux-gnu, and aarch64-linux-gnu. Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Diffstat (limited to 'intl')
-rw-r--r--intl/dcigettext.c3
-rw-r--r--intl/finddomain.c2
-rw-r--r--intl/loadmsgcat.c1
-rw-r--r--intl/localealias.c15
4 files changed, 12 insertions, 9 deletions
diff --git a/intl/dcigettext.c b/intl/dcigettext.c
index 64de9d5..7886ac9 100644
--- a/intl/dcigettext.c
+++ b/intl/dcigettext.c
@@ -1667,7 +1667,8 @@ mempcpy (void *dest, const void *src, size_t n)
#ifdef _LIBC
/* If we want to free all resources we have to do some work at
program's end. */
-libc_freeres_fn (free_mem)
+void
+__intl_freemem (void)
{
void *old;
diff --git a/intl/finddomain.c b/intl/finddomain.c
index 5f2508b..6a8b239 100644
--- a/intl/finddomain.c
+++ b/intl/finddomain.c
@@ -185,7 +185,7 @@ out:
#ifdef _LIBC
/* This is called from iconv/gconv_db.c's free_mem, as locales must
be freed before freeing gconv steps arrays. */
-void __libc_freeres_fn_section
+void
_nl_finddomain_subfreeres (void)
{
struct loaded_l10nfile *runp = _nl_loaded_domains;
diff --git a/intl/loadmsgcat.c b/intl/loadmsgcat.c
index 9924897..94a362f 100644
--- a/intl/loadmsgcat.c
+++ b/intl/loadmsgcat.c
@@ -1284,7 +1284,6 @@ _nl_load_domain (struct loaded_l10nfile *domain_file,
#ifdef _LIBC
void
-__libc_freeres_fn_section
_nl_unload_domain (struct loaded_domain *domain)
{
size_t i;
diff --git a/intl/localealias.c b/intl/localealias.c
index 42e4569..ea4f48b 100644
--- a/intl/localealias.c
+++ b/intl/localealias.c
@@ -126,14 +126,10 @@ struct alias_map
};
-#ifndef _LIBC
-# define libc_freeres_ptr(decl) decl
-#endif
-
-libc_freeres_ptr (static char *string_space);
+static char *string_space;
static size_t string_space_act;
static size_t string_space_max;
-libc_freeres_ptr (static struct alias_map *map);
+static struct alias_map *map;
static size_t nmap;
static size_t maxmap;
@@ -439,3 +435,10 @@ alias_compare (const struct alias_map *map1, const struct alias_map *map2)
return c1 - c2;
#endif
}
+
+void
+__libc_localealias_freemem (void)
+{
+ free (string_space);
+ free (map);
+}