]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
journal-remote: destroy RemoteServer object by using _cleanup_ attribute
authorYu Watanabe <watanabe.yu+github@gmail.com>
Mon, 26 Nov 2018 08:50:52 +0000 (17:50 +0900)
committerYu Watanabe <watanabe.yu+github@gmail.com>
Sat, 8 Dec 2018 09:09:40 +0000 (18:09 +0900)
src/journal-remote/journal-remote-main.c
src/journal-remote/journal-remote.c
src/journal-remote/journal-remote.h

index b82d4b4a1b61e254194f5ddbfbf1793b650359a4..655c099d67833f726b9f61d1cdc0ca9f60a52a15 100644 (file)
@@ -1062,7 +1062,7 @@ static int load_certificates(char **key, char **cert, char **trust) {
 }
 
 int main(int argc, char **argv) {
-        RemoteServer s = {};
+        _cleanup_(journal_remote_server_destroy) RemoteServer s = {};
         int r;
         _cleanup_free_ char *key = NULL, *cert = NULL, *trust = NULL;
 
@@ -1126,8 +1126,6 @@ int main(int argc, char **argv) {
                    "STATUS=Shutting down after writing %" PRIu64 " entries...", s.event_count);
         log_info("Finishing after writing %" PRIu64 " entries", s.event_count);
 
-        journal_remote_server_destroy(&s);
-
         free(arg_key);
         free(arg_cert);
         free(arg_trust);
index 1f3cdb932f7593c292817d4134c6751b7ef48380..3c0916c4383236611dc037d77e1f98fa2f865f60 100644 (file)
@@ -346,7 +346,7 @@ static void MHDDaemonWrapper_free(MHDDaemonWrapper *d) {
 }
 #endif
 
-RemoteServer* journal_remote_server_destroy(RemoteServer *s) {
+void journal_remote_server_destroy(RemoteServer *s) {
         size_t i;
 
 #if HAVE_MICROHTTPD
@@ -370,7 +370,6 @@ RemoteServer* journal_remote_server_destroy(RemoteServer *s) {
                 journal_remote_server_global = NULL;
 
         /* fds that we're listening on remain open... */
-        return NULL;
 }
 
 /**********************************************************************
index e083ea9c74975a208cefad4fd9881f1218463534..4c25d43abf955a6d352fda60aca0d3836dcb6f01 100644 (file)
@@ -62,4 +62,4 @@ int journal_remote_handle_raw_source(
                 uint32_t revents,
                 RemoteServer *s);
 
-RemoteServer* journal_remote_server_destroy(RemoteServer *s);
+void journal_remote_server_destroy(RemoteServer *s);