|
|
da2bf9 |
From 520ff5394187a0d6cb0cb40251f6e8e997ccdd0e Mon Sep 17 00:00:00 2001
|
|
|
da2bf9 |
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
|
|
|
da2bf9 |
Date: Wed, 7 Apr 2021 17:54:49 +0200
|
|
|
da2bf9 |
Subject: [PATCH] tmpfiles: rework condition check
|
|
|
da2bf9 |
|
|
|
da2bf9 |
(!a && b) || (a && c) is replaced by (a ? c : b).
|
|
|
da2bf9 |
|
|
|
da2bf9 |
path_startswith() != NULL is need to avoid type warning.
|
|
|
da2bf9 |
|
|
|
da2bf9 |
(cherry picked from commit 875e7b25d84a111755dab79241c9e64e44836910)
|
|
|
da2bf9 |
|
|
|
da2bf9 |
Related: #1944468
|
|
|
da2bf9 |
---
|
|
|
da2bf9 |
src/tmpfiles/tmpfiles.c | 11 ++++++++---
|
|
|
da2bf9 |
1 file changed, 8 insertions(+), 3 deletions(-)
|
|
|
da2bf9 |
|
|
|
da2bf9 |
diff --git a/src/tmpfiles/tmpfiles.c b/src/tmpfiles/tmpfiles.c
|
|
|
da2bf9 |
index 1aeeed0d2e..50fada99dd 100644
|
|
|
da2bf9 |
--- a/src/tmpfiles/tmpfiles.c
|
|
|
da2bf9 |
+++ b/src/tmpfiles/tmpfiles.c
|
|
|
da2bf9 |
@@ -2705,7 +2705,11 @@ static int read_config_file(char **config_dirs, const char *fn, bool ignore_enoe
|
|
|
da2bf9 |
for (size_t nj = 0; nj < ja->count; nj++) {
|
|
|
da2bf9 |
Item *j = ja->items + nj;
|
|
|
da2bf9 |
|
|
|
da2bf9 |
- if (!IN_SET(j->type, CREATE_DIRECTORY, TRUNCATE_DIRECTORY, CREATE_SUBVOLUME, CREATE_SUBVOLUME_INHERIT_QUOTA, CREATE_SUBVOLUME_NEW_QUOTA))
|
|
|
da2bf9 |
+ if (!IN_SET(j->type, CREATE_DIRECTORY,
|
|
|
da2bf9 |
+ TRUNCATE_DIRECTORY,
|
|
|
da2bf9 |
+ CREATE_SUBVOLUME,
|
|
|
da2bf9 |
+ CREATE_SUBVOLUME_INHERIT_QUOTA,
|
|
|
da2bf9 |
+ CREATE_SUBVOLUME_NEW_QUOTA))
|
|
|
da2bf9 |
continue;
|
|
|
da2bf9 |
|
|
|
da2bf9 |
if (path_equal(j->path, i->path)) {
|
|
|
da2bf9 |
@@ -2713,8 +2717,9 @@ static int read_config_file(char **config_dirs, const char *fn, bool ignore_enoe
|
|
|
da2bf9 |
break;
|
|
|
da2bf9 |
}
|
|
|
da2bf9 |
|
|
|
da2bf9 |
- if ((!candidate_item && path_startswith(i->path, j->path)) ||
|
|
|
da2bf9 |
- (candidate_item && path_startswith(j->path, candidate_item->path) && (fnmatch(i->path, j->path, FNM_PATHNAME | FNM_PERIOD) == 0)))
|
|
|
da2bf9 |
+ if (candidate_item
|
|
|
da2bf9 |
+ ? (path_startswith(j->path, candidate_item->path) && fnmatch(i->path, j->path, FNM_PATHNAME | FNM_PERIOD) == 0)
|
|
|
da2bf9 |
+ : path_startswith(i->path, j->path) != NULL)
|
|
|
da2bf9 |
candidate_item = j;
|
|
|
da2bf9 |
}
|
|
|
da2bf9 |
|