You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
123 lines
3.6 KiB
123 lines
3.6 KiB
commit 198660741b23ec9defb19e22951d4a721de603c8 |
|
Author: Adhemerval Zanella <adhemerval.zanella@linaro.org> |
|
Date: Tue Jul 20 14:04:51 2021 -0300 |
|
|
|
elf: Add _dl_audit_objclose |
|
|
|
It consolidates the code required to call la_objclose audit |
|
callback. |
|
|
|
Checked on x86_64-linux-gnu, i686-linux-gnu, and aarch64-linux-gnu. |
|
|
|
Reviewed-by: Florian Weimer <fweimer@redhat.com> |
|
(cherry picked from commit 311c9ee54ea963ff69bd3a2e6981c37e893b4c3e) |
|
|
|
diff --git a/elf/dl-audit.c b/elf/dl-audit.c |
|
index 5682427220569d90..cb1c3de93cba447b 100644 |
|
--- a/elf/dl-audit.c |
|
+++ b/elf/dl-audit.c |
|
@@ -85,3 +85,24 @@ _dl_audit_objopen (struct link_map *l, Lmid_t nsid) |
|
afct = afct->next; |
|
} |
|
} |
|
+ |
|
+void |
|
+_dl_audit_objclose (struct link_map *l) |
|
+{ |
|
+ if (__glibc_likely (GLRO(dl_naudit) == 0) |
|
+ || GL(dl_ns)[l->l_ns]._ns_loaded->l_auditing) |
|
+ return; |
|
+ |
|
+ struct audit_ifaces *afct = GLRO(dl_audit); |
|
+ for (unsigned int cnt = 0; cnt < GLRO(dl_naudit); ++cnt) |
|
+ { |
|
+ if (afct->objclose != NULL) |
|
+ { |
|
+ struct auditstate *state= link_map_audit_state (l, cnt); |
|
+ /* Return value is ignored. */ |
|
+ afct->objclose (&state->cookie); |
|
+ } |
|
+ |
|
+ afct = afct->next; |
|
+ } |
|
+} |
|
diff --git a/elf/dl-close.c b/elf/dl-close.c |
|
index 5a8cc9e7cb5186cc..985cd4e2821436af 100644 |
|
--- a/elf/dl-close.c |
|
+++ b/elf/dl-close.c |
|
@@ -260,9 +260,6 @@ _dl_close_worker (struct link_map *map, bool force) |
|
_dl_sort_maps (maps, nloaded, (nsid == LM_ID_BASE), true); |
|
|
|
/* Call all termination functions at once. */ |
|
-#ifdef SHARED |
|
- bool do_audit = GLRO(dl_naudit) > 0 && !ns->_ns_loaded->l_auditing; |
|
-#endif |
|
bool unload_any = false; |
|
bool scope_mem_left = false; |
|
unsigned int unload_global = 0; |
|
@@ -296,22 +293,7 @@ _dl_close_worker (struct link_map *map, bool force) |
|
|
|
#ifdef SHARED |
|
/* Auditing checkpoint: we remove an object. */ |
|
- if (__glibc_unlikely (do_audit)) |
|
- { |
|
- struct audit_ifaces *afct = GLRO(dl_audit); |
|
- for (unsigned int cnt = 0; cnt < GLRO(dl_naudit); ++cnt) |
|
- { |
|
- if (afct->objclose != NULL) |
|
- { |
|
- struct auditstate *state |
|
- = link_map_audit_state (imap, cnt); |
|
- /* Return value is ignored. */ |
|
- (void) afct->objclose (&state->cookie); |
|
- } |
|
- |
|
- afct = afct->next; |
|
- } |
|
- } |
|
+ _dl_audit_objclose (imap); |
|
#endif |
|
|
|
/* This object must not be used anymore. */ |
|
diff --git a/elf/dl-fini.c b/elf/dl-fini.c |
|
index c683884c355dfd52..b789cfb9f2ac6c85 100644 |
|
--- a/elf/dl-fini.c |
|
+++ b/elf/dl-fini.c |
|
@@ -146,21 +146,7 @@ _dl_fini (void) |
|
|
|
#ifdef SHARED |
|
/* Auditing checkpoint: another object closed. */ |
|
- if (!do_audit && __builtin_expect (GLRO(dl_naudit) > 0, 0)) |
|
- { |
|
- struct audit_ifaces *afct = GLRO(dl_audit); |
|
- for (unsigned int cnt = 0; cnt < GLRO(dl_naudit); ++cnt) |
|
- { |
|
- if (afct->objclose != NULL) |
|
- { |
|
- struct auditstate *state |
|
- = link_map_audit_state (l, cnt); |
|
- /* Return value is ignored. */ |
|
- (void) afct->objclose (&state->cookie); |
|
- } |
|
- afct = afct->next; |
|
- } |
|
- } |
|
+ _dl_audit_objclose (l); |
|
#endif |
|
} |
|
|
|
diff --git a/sysdeps/generic/ldsodefs.h b/sysdeps/generic/ldsodefs.h |
|
index 1f212a18d7bfc440..982f23c0287955fe 100644 |
|
--- a/sysdeps/generic/ldsodefs.h |
|
+++ b/sysdeps/generic/ldsodefs.h |
|
@@ -1394,6 +1394,10 @@ void _dl_audit_activity_nsid (Lmid_t nsid, int action) |
|
namespace identification NSID. */ |
|
void _dl_audit_objopen (struct link_map *l, Lmid_t nsid) |
|
attribute_hidden; |
|
+ |
|
+/* Call the la_objclose from the audit modules for the link_map L. */ |
|
+void _dl_audit_objclose (struct link_map *l) |
|
+ attribute_hidden; |
|
#endif /* SHARED */ |
|
|
|
#if PTHREAD_IN_LIBC && defined SHARED
|
|
|