c5d972
commit 0b98a8748759e88b58927882a8714109abe0a2d6
c5d972
Author: Adhemerval Zanella <adhemerval.zanella@linaro.org>
c5d972
Date:   Thu Jul 22 17:10:57 2021 -0300
c5d972
c5d972
    elf: Add _dl_audit_preinit
c5d972
    
c5d972
    It consolidates the code required to call la_preinit audit
c5d972
    callback.
c5d972
    
c5d972
    Checked on x86_64-linux-gnu, i686-linux-gnu, and aarch64-linux-gnu.
c5d972
    
c5d972
    Reviewed-by: Florian Weimer <fweimer@redhat.com>
c5d972
c5d972
Conflicts:
c5d972
	csu/libc-start.c
c5d972
		Rework to existing init call code.
c5d972
c5d972
diff --git a/csu/libc-start.c b/csu/libc-start.c
c5d972
index fd0f8640eaeae34c..ae703cfa620163fd 100644
c5d972
--- a/csu/libc-start.c
c5d972
+++ b/csu/libc-start.c
c5d972
@@ -265,32 +265,20 @@ LIBC_START_MAIN (int (*main) (int, char **, char ** MAIN_AUXVEC_DECL),
c5d972
 #ifdef SHARED
c5d972
   if (__builtin_expect (GLRO(dl_debug_mask) & DL_DEBUG_IMPCALLS, 0))
c5d972
     GLRO(dl_debug_printf) ("\ninitialize program: %s\n\n", argv[0]);
c5d972
-#endif
c5d972
+
c5d972
   if (init)
c5d972
     (*init) (argc, argv, __environ MAIN_AUXVEC_PARAM);
c5d972
 
c5d972
-#ifdef SHARED
c5d972
   /* Auditing checkpoint: we have a new object.  */
c5d972
-  if (__glibc_unlikely (GLRO(dl_naudit) > 0))
c5d972
-    {
c5d972
-      struct audit_ifaces *afct = GLRO(dl_audit);
c5d972
-      struct link_map *head = GL(dl_ns)[LM_ID_BASE]._ns_loaded;
c5d972
-      for (unsigned int cnt = 0; cnt < GLRO(dl_naudit); ++cnt)
c5d972
-	{
c5d972
-	  if (afct->preinit != NULL)
c5d972
-	    afct->preinit (&link_map_audit_state (head, cnt)->cookie);
c5d972
-
c5d972
-	  afct = afct->next;
c5d972
-	}
c5d972
-    }
c5d972
-#endif
c5d972
+  _dl_audit_preinit (GL(dl_ns)[LM_ID_BASE]._ns_loaded);
c5d972
 
c5d972
-#ifdef SHARED
c5d972
   if (__glibc_unlikely (GLRO(dl_debug_mask) & DL_DEBUG_IMPCALLS))
c5d972
     GLRO(dl_debug_printf) ("\ntransferring control: %s\n\n", argv[0]);
c5d972
-#endif
c5d972
 
c5d972
-#ifndef SHARED
c5d972
+#else /* !SHARED */
c5d972
+  if (init)
c5d972
+    (*init) (argc, argv, __environ MAIN_AUXVEC_PARAM);
c5d972
+
c5d972
   _dl_debug_initialize (0, LM_ID_BASE);
c5d972
 #endif
c5d972
 #ifdef HAVE_CLEANUP_JMP_BUF
c5d972
diff --git a/elf/Versions b/elf/Versions
c5d972
index c5d4342cf1f5124c..35ac181bdb099af8 100644
c5d972
--- a/elf/Versions
c5d972
+++ b/elf/Versions
c5d972
@@ -59,7 +59,7 @@ ld {
c5d972
     _dl_argv; _dl_find_dso_for_object; _dl_get_tls_static_info;
c5d972
     _dl_deallocate_tls; _dl_make_stack_executable;
c5d972
     _dl_rtld_di_serinfo; _dl_starting_up; _dl_fatal_printf;
c5d972
-    _dl_audit_symbind_alt;
c5d972
+    _dl_audit_symbind_alt; _dl_audit_preinit;
c5d972
     _rtld_global; _rtld_global_ro;
c5d972
 
c5d972
     # Only here for gdb while a better method is developed.
c5d972
diff --git a/elf/dl-audit.c b/elf/dl-audit.c
c5d972
index a21530f30bc5524b..0b6fac8e48877c93 100644
c5d972
--- a/elf/dl-audit.c
c5d972
+++ b/elf/dl-audit.c
c5d972
@@ -108,6 +108,21 @@ _dl_audit_objclose (struct link_map *l)
c5d972
     }
c5d972
 }
c5d972
 
c5d972
+void
c5d972
+_dl_audit_preinit (struct link_map *l)
c5d972
+{
c5d972
+  if (__glibc_likely (GLRO(dl_naudit) == 0))
c5d972
+    return;
c5d972
+
c5d972
+  struct audit_ifaces *afct = GLRO(dl_audit);
c5d972
+  for (unsigned int cnt = 0; cnt < GLRO(dl_naudit); ++cnt)
c5d972
+    {
c5d972
+      if (afct->preinit != NULL)
c5d972
+	afct->preinit (&link_map_audit_state (l, cnt)->cookie);
c5d972
+      afct = afct->next;
c5d972
+    }
c5d972
+}
c5d972
+
c5d972
 void
c5d972
 _dl_audit_symbind_alt (struct link_map *l, const ElfW(Sym) *ref, void **value,
c5d972
 		       lookup_t result)
c5d972
diff --git a/sysdeps/generic/ldsodefs.h b/sysdeps/generic/ldsodefs.h
c5d972
index fa55c3bde10de52e..03676b474c3d37a3 100644
c5d972
--- a/sysdeps/generic/ldsodefs.h
c5d972
+++ b/sysdeps/generic/ldsodefs.h
c5d972
@@ -1295,6 +1295,9 @@ void _dl_audit_objopen (struct link_map *l, Lmid_t nsid)
c5d972
 void _dl_audit_objclose (struct link_map *l)
c5d972
   attribute_hidden;
c5d972
 
c5d972
+/* Call the la_preinit from the audit modules for the link_map L.  */
c5d972
+void _dl_audit_preinit (struct link_map *l);
c5d972
+
c5d972
 /* Call the la_symbind{32,64} from the audit modules for the link_map L.  */
c5d972
 void _dl_audit_symbind (struct link_map *l, struct reloc_result *reloc_result,
c5d972
 			const ElfW(Sym) *defsym, DL_FIXUP_VALUE_TYPE *value,