]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
tree-wide: explicitly ignore return value in a couple more places 28226/head
authorFrantisek Sumsal <frantisek@sumsal.cz>
Sun, 2 Jul 2023 09:39:00 +0000 (11:39 +0200)
committerFrantisek Sumsal <frantisek@sumsal.cz>
Sun, 2 Jul 2023 10:22:45 +0000 (12:22 +0200)
Resolves:
  - CID#1490777
  - CID#1498366
  - CID#1508639
  - CID#1509084
  - CID#1509086
  - CID#1509087

src/basic/process-util.c
src/libsystemd/sd-device/sd-device.c
src/libsystemd/sd-journal/test-mmap-cache.c
src/test/test-execute.c
src/test/test-io-util.c
src/udev/udev-watch.c

index 001002a55e9f8466feabbacc366685080191acaf..6cf882866fd755000da742f85d4921b15009dab2 100644 (file)
@@ -1095,7 +1095,7 @@ pid_t getpid_cached(void) {
          * https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=c579f48edba88380635ab98cb612030e3ed8691e
          */
 
-        __atomic_compare_exchange_n(
+        (void) __atomic_compare_exchange_n(
                         &cached_pid,
                         &current_value,
                         CACHED_PID_BUSY,
index 2c7b97ab4daa08fbd2c65ae160cac35397974029..af39c984d0e518ed09d9e8fd6bf561c825a4191e 100644 (file)
@@ -1013,7 +1013,7 @@ _public_ sd_device *sd_device_get_child_next(sd_device *device, const char **ret
 
         assert(device);
 
-        hashmap_iterate(device->children, &device->children_iterator, (void**) &child, (const void**) ret_suffix);
+        (void) hashmap_iterate(device->children, &device->children_iterator, (void**) &child, (const void**) ret_suffix);
         return child;
 }
 
index e3dfbada3273a4a93645e5a3d050ef4e3e79d26f..bc063381b2ba1c74d8cab4000f069e0eed40f4dc 100644 (file)
@@ -21,17 +21,17 @@ int main(int argc, char *argv[]) {
 
         x = mkostemp_safe(px);
         assert_se(x >= 0);
-        unlink(px);
+        (void) unlink(px);
 
         assert_se(fx = mmap_cache_add_fd(m, x, PROT_READ));
 
         y = mkostemp_safe(py);
         assert_se(y >= 0);
-        unlink(py);
+        (void) unlink(py);
 
         z = mkostemp_safe(pz);
         assert_se(z >= 0);
-        unlink(pz);
+        (void) unlink(pz);
 
         r = mmap_cache_fd_get(fx, 0, false, 1, 2, NULL, &p);
         assert_se(r >= 0);
index 164c1bc0337e6ac86bf3c5f91c1d69fc86be819b..27d2af78f36f598dad5ae91b8ea00b689bd3fa11 100644 (file)
@@ -419,7 +419,7 @@ static void test_exec_privatetmp(Manager *m) {
         test(m, "exec-privatetmp-no.service", 0, CLD_EXITED);
         test(m, "exec-privatetmp-disabled-by-prefix.service", can_unshare ? 0 : MANAGER_IS_SYSTEM(m) ? EXIT_FAILURE : EXIT_NAMESPACE, CLD_EXITED);
 
-        unlink("/tmp/test-exec_privatetmp");
+        (void) unlink("/tmp/test-exec_privatetmp");
 }
 
 static void test_exec_privatedevices(Manager *m) {
index fd41a2a4d109f09c1773a88cdd08797b9fd33421..fa3ef865c94731029eca303de8ad8048ebb4090b 100644 (file)
@@ -37,7 +37,7 @@ TEST(sparse_write) {
 
         fd = mkostemp(fn, O_CLOEXEC);
         assert_se(fd >= 0);
-        unlink(fn);
+        (void) unlink(fn);
 
         test_sparse_write_one(fd, test_a, sizeof(test_a));
         test_sparse_write_one(fd, test_b, sizeof(test_b));
index d00226cd7a42c2f7c4b4bf3b5defdecf3c994f7e..58c82794f0f8384ab1c984fd5e6803f4d9819965 100644 (file)
@@ -51,7 +51,7 @@ int udev_watch_restore(int inotify_fd) {
 
         assert(inotify_fd >= 0);
 
-        rm_rf("/run/udev/watch.old", REMOVE_ROOT);
+        (void) rm_rf("/run/udev/watch.old", REMOVE_ROOT);
 
         if (rename("/run/udev/watch", "/run/udev/watch.old") < 0) {
                 if (errno == ENOENT)