]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
tree-wide: use sockaddr_un_unlink() whereever appropriate
authorLennart Poettering <lennart@poettering.net>
Mon, 15 Oct 2018 10:08:30 +0000 (12:08 +0200)
committerLennart Poettering <lennart@poettering.net>
Mon, 15 Oct 2018 17:35:00 +0000 (19:35 +0200)
Let's port everything over.

src/core/dbus.c
src/core/manager.c
src/import/importd.c
src/journal/journald-native.c
src/journal/journald-stream.c
src/journal/journald-syslog.c
src/udev/udev-ctrl.c

index bf5917696e0b9c5732095ba27c583e9a0d2515eb..4b04498423ac3dec7284ed8d4c180ab69f8e3164 100644 (file)
@@ -1012,7 +1012,7 @@ int bus_init_private(Manager *m) {
         }
 
         (void) mkdir_parents_label(sa.un.sun_path, 0755);
-        (void) unlink(sa.un.sun_path);
+        (void) sockaddr_un_unlink(&sa.un);
 
         fd = socket(AF_UNIX, SOCK_STREAM|SOCK_CLOEXEC|SOCK_NONBLOCK, 0);
         if (fd < 0)
index 971faef90a74641243008f3b1340b4cd9ba0758a..e8ade8b7f7c8e60a688863ee992eaab6f20b109b 100644 (file)
@@ -970,7 +970,7 @@ static int manager_setup_cgroups_agent(Manager *m) {
 
                 fd_inc_rcvbuf(fd, CGROUPS_AGENT_RCVBUF_SIZE);
 
-                (void) unlink(sa.un.sun_path);
+                (void) sockaddr_un_unlink(&sa.un);
 
                 /* Only allow root to connect to this socket */
                 RUN_WITH_UMASK(0077)
index 04563fb0981cd99462cfbff20acd5ae57cb401c1..c42b3914aa5fe82074467ae8a7ba1b890fe8f6d4 100644 (file)
@@ -608,7 +608,7 @@ static int manager_new(Manager **ret) {
                 return -errno;
 
         (void) mkdir_parents_label(sa.un.sun_path, 0755);
-        (void) unlink(sa.un.sun_path);
+        (void) sockaddr_un_unlink(&sa.un);
 
         if (bind(m->notify_fd, &sa.sa, SOCKADDR_UN_LEN(sa.un)) < 0)
                 return -errno;
index 44610a4b3b33cba62494759a68dd36702f8920e7..bf1ef64e70699028423ed541988437e27c35f74e 100644 (file)
@@ -453,7 +453,7 @@ int server_open_native_socket(Server*s) {
                 if (s->native_fd < 0)
                         return log_error_errno(errno, "socket() failed: %m");
 
-                (void) unlink(sa.un.sun_path);
+                (void) sockaddr_un_unlink(&sa.un);
 
                 r = bind(s->native_fd, &sa.sa, SOCKADDR_UN_LEN(sa.un));
                 if (r < 0)
index 267e63f4a25b1fc308a02401f830496d3febcdb8..502bd6f51bb9e8cf58da7699924ae18f733a5281 100644 (file)
@@ -793,7 +793,7 @@ int server_open_stdout_socket(Server *s) {
                 if (s->stdout_fd < 0)
                         return log_error_errno(errno, "socket() failed: %m");
 
-                (void) unlink(sa.un.sun_path);
+                (void) sockaddr_un_unlink(&sa.un);
 
                 r = bind(s->stdout_fd, &sa.sa, SOCKADDR_UN_LEN(sa.un));
                 if (r < 0)
index 46cc110f1cc7d76434fd57ceb258cef16404dbba..5bf057f4ce280610c797cfa6a6c048109f00a566 100644 (file)
@@ -457,7 +457,7 @@ int server_open_syslog_socket(Server *s) {
                 if (s->syslog_fd < 0)
                         return log_error_errno(errno, "socket() failed: %m");
 
-                (void) unlink(sa.un.sun_path);
+                (void) sockaddr_un_unlink(&sa.un);
 
                 r = bind(s->syslog_fd, &sa.sa, SOCKADDR_UN_LEN(sa.un));
                 if (r < 0)
index 64024100fe89e1defa1135cadd815d0fa6760872..df9ceef7ce6a98e27cb5934a1883106f4ef79585 100644 (file)
@@ -117,7 +117,7 @@ int udev_ctrl_enable_receiving(struct udev_ctrl *uctrl) {
         if (!uctrl->bound) {
                 err = bind(uctrl->sock, &uctrl->saddr.sa, uctrl->addrlen);
                 if (err < 0 && errno == EADDRINUSE) {
-                        unlink(uctrl->saddr.un.sun_path);
+                        (void) sockaddr_un_unlink(&uctrl->saddr.un);
                         err = bind(uctrl->sock, &uctrl->saddr.sa, uctrl->addrlen);
                 }
 
@@ -148,7 +148,7 @@ int udev_ctrl_cleanup(struct udev_ctrl *uctrl) {
         if (uctrl == NULL)
                 return 0;
         if (uctrl->cleanup_socket)
-                unlink(uctrl->saddr.un.sun_path);
+                sockaddr_un_unlink(&uctrl->saddr.un);
         return 0;
 }