]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
util-lib: use FLAGS_SET() where appropriate
authorLennart Poettering <lennart@poettering.net>
Tue, 2 Apr 2019 12:49:28 +0000 (14:49 +0200)
committerLennart Poettering <lennart@poettering.net>
Tue, 2 Apr 2019 12:54:38 +0000 (14:54 +0200)
src/basic/fs-util.c
src/shared/bus-util.c

index 9f3fd4569111ba8e1700a7b9170fc52bb8e8f69f..d1c06cf12a3a92223ce3ab2f4aa682105a5c32d9 100644 (file)
@@ -1411,9 +1411,9 @@ int open_parent(const char *path, int flags, mode_t mode) {
         /* Let's insist on O_DIRECTORY since the parent of a file or directory is a directory. Except if we open an
          * O_TMPFILE file, because in that case we are actually create a regular file below the parent directory. */
 
-        if ((flags & O_PATH) == O_PATH)
+        if (FLAGS_SET(flags, O_PATH))
                 flags |= O_DIRECTORY;
-        else if ((flags & O_TMPFILE) != O_TMPFILE)
+        else if (!FLAGS_SET(flags, O_TMPFILE))
                 flags |= O_DIRECTORY|O_RDONLY;
 
         fd = open(parent, flags, mode);
index 1911dd0ce642f875db511e9b5c56da66752d1942..00120e987414e4d4c7b683d71cbecea7d85414ac 100644 (file)
@@ -756,7 +756,7 @@ static int bus_print_property(const char *name, const char *expected_value, sd_b
 
                         if ((u & NAMESPACE_FLAGS_ALL) == 0)
                                 result = "yes";
-                        else if ((u & NAMESPACE_FLAGS_ALL) == NAMESPACE_FLAGS_ALL)
+                        else if (FLAGS_SET(u, NAMESPACE_FLAGS_ALL))
                                 result = "no";
                         else {
                                 r = namespace_flags_to_string(u, &s);