diff --git a/src/basic/string-util.h b/src/basic/string-util.h index 9cf11198b1..b27ad511bd 100644 --- a/src/basic/string-util.h +++ b/src/basic/string-util.h @@ -57,6 +57,12 @@ static inline const char *empty_to_dash(const char *str) { return isempty(str) ? "-" : str; } +static inline bool empty_or_dash(const char *str) { + return !str || + str[0] == 0 || + (str[0] == '-' && str[1] == 0); +} + static inline char *startswith(const char *s, const char *prefix) { size_t l; diff --git a/src/import/export.c b/src/import/export.c index 4566a66a15..ffd9347e92 100644 --- a/src/import/export.c +++ b/src/import/export.c @@ -78,7 +78,7 @@ static int export_tar(int argc, char *argv[], void *userdata) { if (argc >= 3) path = argv[2]; - if (isempty(path) || streq(path, "-")) + if (empty_or_dash(path)) path = NULL; determine_compression_from_filename(path); @@ -155,7 +155,7 @@ static int export_raw(int argc, char *argv[], void *userdata) { if (argc >= 3) path = argv[2]; - if (isempty(path) || streq(path, "-")) + if (empty_or_dash(path)) path = NULL; determine_compression_from_filename(path); diff --git a/src/import/import-fs.c b/src/import/import-fs.c index 974c0f5536..9f3ef0826c 100644 --- a/src/import/import-fs.c +++ b/src/import/import-fs.c @@ -117,14 +117,14 @@ static int import_fs(int argc, char *argv[], void *userdata) { if (argc >= 2) path = argv[1]; - if (isempty(path) || streq(path, "-")) + if (empty_or_dash(path)) path = NULL; if (argc >= 3) local = argv[2]; else if (path) local = basename(path); - if (isempty(local) || streq(local, "-")) + if (empty_or_dash(local)) local = NULL; if (local) { diff --git a/src/import/import.c b/src/import/import.c index e3a1ae8a8b..4c9603fbdc 100644 --- a/src/import/import.c +++ b/src/import/import.c @@ -49,14 +49,14 @@ static int import_tar(int argc, char *argv[], void *userdata) { if (argc >= 2) path = argv[1]; - if (isempty(path) || streq(path, "-")) + if (empty_or_dash(path)) path = NULL; if (argc >= 3) local = argv[2]; else if (path) local = basename(path); - if (isempty(local) || streq(local, "-")) + if (empty_or_dash(local)) local = NULL; if (local) { @@ -145,14 +145,14 @@ static int import_raw(int argc, char *argv[], void *userdata) { if (argc >= 2) path = argv[1]; - if (isempty(path) || streq(path, "-")) + if (empty_or_dash(path)) path = NULL; if (argc >= 3) local = argv[2]; else if (path) local = basename(path); - if (isempty(local) || streq(local, "-")) + if (empty_or_dash(local)) local = NULL; if (local) { diff --git a/src/import/pull.c b/src/import/pull.c index 68b1221b72..6342f7d7f1 100644 --- a/src/import/pull.c +++ b/src/import/pull.c @@ -64,7 +64,7 @@ static int pull_tar(int argc, char *argv[], void *userdata) { local = l; } - if (isempty(local) || streq(local, "-")) + if (empty_or_dash(local)) local = NULL; if (local) { @@ -151,7 +151,7 @@ static int pull_raw(int argc, char *argv[], void *userdata) { local = l; } - if (isempty(local) || streq(local, "-")) + if (empty_or_dash(local)) local = NULL; if (local) { diff --git a/src/locale/keymap-util.c b/src/locale/keymap-util.c index 17540089ba..18affb79ec 100644 --- a/src/locale/keymap-util.c +++ b/src/locale/keymap-util.c @@ -33,7 +33,7 @@ static bool startswith_comma(const char *s, const char *prefix) { } static const char* strnulldash(const char *s) { - return isempty(s) || streq(s, "-") ? NULL : s; + return empty_or_dash(s) ? NULL : s; } static const char* systemd_kbd_model_map(void) { diff --git a/src/machine/machinectl.c b/src/machine/machinectl.c index c8607bcec3..d6e9c8c03c 100644 --- a/src/machine/machinectl.c +++ b/src/machine/machinectl.c @@ -1994,13 +1994,7 @@ static int transfer_image_common(sd_bus *bus, sd_bus_message *m) { } static const char *nullify_dash(const char *p) { - if (isempty(p)) - return NULL; - - if (streq(p, "-")) - return NULL; - - return p; + return empty_or_dash(p) ? NULL : p; } static int import_tar(int argc, char *argv[], void *userdata) { @@ -2230,7 +2224,7 @@ static int export_tar(int argc, char *argv[], void *userdata) { if (argc >= 3) path = argv[2]; - if (isempty(path) || streq(path, "-")) + if (empty_or_dash(path)) path = NULL; if (path) { @@ -2280,7 +2274,7 @@ static int export_raw(int argc, char *argv[], void *userdata) { if (argc >= 3) path = argv[2]; - if (isempty(path) || streq(path, "-")) + if (empty_or_dash(path)) path = NULL; if (path) { @@ -2338,7 +2332,7 @@ static int pull_tar(int argc, char *argv[], void *userdata) { local = l; } - if (isempty(local) || streq(local, "-")) + if (empty_or_dash(local)) local = NULL; if (local) { @@ -2402,7 +2396,7 @@ static int pull_raw(int argc, char *argv[], void *userdata) { local = l; } - if (isempty(local) || streq(local, "-")) + if (empty_or_dash(local)) local = NULL; if (local) { diff --git a/src/sysusers/sysusers.c b/src/sysusers/sysusers.c index 9a85a20be3..843c3837d1 100644 --- a/src/sysusers/sysusers.c +++ b/src/sysusers/sysusers.c @@ -1411,7 +1411,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer) { "[%s:%u] Unknown command type '%c'.", fname, line, action[0]); /* Verify name */ - if (isempty(name) || streq(name, "-")) + if (empty_or_dash(name)) name = mfree(name); if (name) { @@ -1426,7 +1426,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer) { } /* Verify id */ - if (isempty(id) || streq(id, "-")) + if (empty_or_dash(id)) id = mfree(id); if (id) { @@ -1437,7 +1437,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer) { } /* Verify description */ - if (isempty(description) || streq(description, "-")) + if (empty_or_dash(description)) description = mfree(description); if (description) { @@ -1453,7 +1453,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer) { } /* Verify home */ - if (isempty(home) || streq(home, "-")) + if (empty_or_dash(home)) home = mfree(home); if (home) { @@ -1469,7 +1469,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer) { } /* Verify shell */ - if (isempty(shell) || streq(shell, "-")) + if (empty_or_dash(shell)) shell = mfree(shell); if (shell) { diff --git a/src/tmpfiles/tmpfiles.c b/src/tmpfiles/tmpfiles.c index 84e104f9b5..d9d1cc1c1a 100644 --- a/src/tmpfiles/tmpfiles.c +++ b/src/tmpfiles/tmpfiles.c @@ -2507,7 +2507,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer, bool return -EIO; } - if (!isempty(buffer) && !streq(buffer, "-")) { + if (!empty_or_dash(buffer)) { i.argument = strdup(buffer); if (!i.argument) return log_oom(); @@ -2704,7 +2704,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer, bool free_and_replace(i.path, p); } - if (!isempty(user) && !streq(user, "-")) { + if (!empty_or_dash(user)) { const char *u = user; r = get_user_creds(&u, &i.uid, NULL, NULL, NULL, USER_CREDS_ALLOW_MISSING); @@ -2716,7 +2716,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer, bool i.uid_set = true; } - if (!isempty(group) && !streq(group, "-")) { + if (!empty_or_dash(group)) { const char *g = group; r = get_group_creds(&g, &i.gid, USER_CREDS_ALLOW_MISSING); @@ -2729,7 +2729,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer, bool i.gid_set = true; } - if (!isempty(mode) && !streq(mode, "-")) { + if (!empty_or_dash(mode)) { const char *mm = mode; unsigned m; @@ -2749,7 +2749,7 @@ static int parse_line(const char *fname, unsigned line, const char *buffer, bool } else i.mode = IN_SET(i.type, CREATE_DIRECTORY, TRUNCATE_DIRECTORY, CREATE_SUBVOLUME, CREATE_SUBVOLUME_INHERIT_QUOTA, CREATE_SUBVOLUME_NEW_QUOTA) ? 0755 : 0644; - if (!isempty(age) && !streq(age, "-")) { + if (!empty_or_dash(age)) { const char *a = age; if (*a == '~') {