confd,statd: fix invalid option to openlog()

LOG_DAEMON is a facility not an option. Replace with NDELAY option and set facility properly in call to openlog() instead. Signed-off-by: Joachim Wiberg <[email protected]>
kernelkit · Aug 30, 2024 · 7d95da7 · 7d95da7
1 parent cf1c4c3
commit 7d95da7
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/confd/src/core.c b/src/confd/src/core.c
@@ -85,9 +85,9 @@ int core_post_hook(sr_session_ctx_t *session, uint32_t sub_id, const char *modul
 
 int sr_plugin_init_cb(sr_session_ctx_t *session, void **priv)
 {
-	int log_opts = LOG_PID | LOG_DAEMON;
+	int log_opts = LOG_PID | LOG_NDELAY;
 	int rc = SR_ERR_SYS;
-	char *env;
+	const char *env;
 
 	/* Convert into command line option+SIGUSR1 when converting to standalone confd */
 	env = getenv("DEBUG");
@@ -96,7 +96,7 @@ int sr_plugin_init_cb(sr_session_ctx_t *session, void **priv)
 		debug = 1;
 	}
 
-	openlog("confd", log_opts, 0);
+	openlog("confd", log_opts, LOG_DAEMON);
 
 	/* Save context with default running config datastore for all our models */
 	*priv = (void *)&confd;

diff --git a/src/statd/statd.c b/src/statd/statd.c
@@ -535,10 +535,10 @@ static int sub_to_container(struct statd *statd)
 int main(int argc, char *argv[])
 {
 	struct ev_signal sigint_watcher, sigusr1_watcher;
-	int log_opts = LOG_PID | LOG_DAEMON;
+	int log_opts = LOG_PID | LOG_NDELAY;
 	struct statd statd = {};
 	sr_conn_ctx_t *sr_conn;
-	char *env;
+	const char *env;
 	int err;
 
 	env = getenv("DEBUG");
@@ -547,7 +547,7 @@ int main(int argc, char *argv[])
 		debug = 1;
 	}
 
-	openlog("statd", log_opts, 0);
+	openlog("statd", log_opts, LOG_DAEMON);
 
 	TAILQ_INIT(&statd.subs);
 	statd.ev_loop = EV_DEFAULT;