safe_exec exits the child process directly on execve failure

The child side of safe_exec calls exit(1) after a failed execve, which
can run atexit handlers and flush shared stdio buffers. The comment
notes this should “error instead of exit.”

Updated safe_exec in the audit daemon to terminate failing child
processes with _exit(EXIT_FAILURE) rather than exit(1), preventing
execution of atexit handlers and aligning shutdown behavior with
POSIX recommendations.

Author: stevegrubb

src/auditd-event.c

@@ -1483,7 +1483,7 @@ static pid_t safe_exec(const char *exe)
 	argv[1] = NULL;
 	execve(exe, argv, NULL);
 	audit_msg(LOG_ALERT, "Audit daemon failed to exec %s", exe);
-	exit(1); /* FIXME: Maybe this should error instead of exit */
+	_exit(EXIT_FAILURE); // Avoid running the atexit handlers
 }
 
 static void reconfigure(struct auditd_event *e)