diff --git a/src/shared/specifier.c b/src/shared/specifier.c index a02012407ba..df52f8cf818 100644 --- a/src/shared/specifier.c +++ b/src/shared/specifier.c @@ -269,7 +269,9 @@ int specifier_architecture(char specifier, const void *data, const char *root, c } /* Note: fields in /etc/os-release might quite possibly be missing, even if everything is entirely valid - * otherwise. We'll return an empty value or NULL in that case from the functions below. */ + * otherwise. We'll return an empty value or NULL in that case from the functions below. But if the + * os-release file is missing, we'll return -ENOENT. This means that something is seriously wrong with the + * installation. */ int specifier_os_id(char specifier, const void *data, const char *root, const void *userdata, char **ret) { assert(ret); diff --git a/src/test/test-specifier.c b/src/test/test-specifier.c index 231625de958..5ece520e9b4 100644 --- a/src/test/test-specifier.c +++ b/src/test/test-specifier.c @@ -138,4 +138,18 @@ TEST(specifiers) { } } +TEST(specifiers_missing_data_ok) { + _cleanup_free_ char *resolved = NULL; + + assert_se(setenv("SYSTEMD_OS_RELEASE", "/dev/null", 1) == 0); + assert_se(specifier_printf("%A-%B-%M-%o-%w-%W", SIZE_MAX, specifier_table, NULL, NULL, &resolved) >= 0); + assert_se(streq(resolved, "-----")); + + assert_se(setenv("SYSTEMD_OS_RELEASE", "/nosuchfileordirectory", 1) == 0); + assert_se(specifier_printf("%A-%B-%M-%o-%w-%W", SIZE_MAX, specifier_table, NULL, NULL, &resolved) == -ENOENT); + assert_se(streq(resolved, "-----")); + + assert_se(unsetenv("SYSTEMD_OS_RELEASE") == 0); +} + DEFINE_TEST_MAIN(LOG_DEBUG);