diff options
author | Luca Boccassi <bluca@debian.org> | 2022-10-11 18:31:06 +0100 |
---|---|---|
committer | Luca Boccassi <bluca@debian.org> | 2022-10-12 09:57:24 +0100 |
commit | 192a9b70c2bee93666da1b1bbbf529d511180961 (patch) | |
tree | d0a156dba008378abc3baa4007a5421cd459751d /src/basic/os-util.c | |
parent | ac6086fdf63f1c2b26baa955d13faa21afb2aeb1 (diff) | |
download | systemd-192a9b70c2bee93666da1b1bbbf529d511180961.tar.gz |
os-util: split out extension-release xattr helper
Diffstat (limited to 'src/basic/os-util.c')
-rw-r--r-- | src/basic/os-util.c | 66 |
1 files changed, 41 insertions, 25 deletions
diff --git a/src/basic/os-util.c b/src/basic/os-util.c index 04e146f648..503db9b5d4 100644 --- a/src/basic/os-util.c +++ b/src/basic/os-util.c @@ -58,6 +58,44 @@ int path_is_extension_tree(const char *path, const char *extension) { return 1; } +static int extension_release_strict_xattr_value(int extension_release_fd, const char *extension_release_dir_path, const char *filename) { + int r; + + assert(extension_release_fd >= 0); + assert(extension_release_dir_path); + assert(filename); + + /* No xattr or cannot parse it? Then skip this. */ + _cleanup_free_ char *extension_release_xattr = NULL; + r = fgetxattr_malloc(extension_release_fd, "user.extension-release.strict", &extension_release_xattr); + if (r < 0) { + if (!ERRNO_IS_XATTR_ABSENT(r)) + return log_debug_errno(r, + "%s/%s: Failed to read 'user.extension-release.strict' extended attribute from file, ignoring: %m", + extension_release_dir_path, filename); + + return log_debug_errno(r, "%s/%s does not have user.extension-release.strict xattr, ignoring.", extension_release_dir_path, filename); + } + + /* Explicitly set to request strict matching? Skip it. */ + r = parse_boolean(extension_release_xattr); + if (r < 0) + return log_debug_errno(r, + "%s/%s: Failed to parse 'user.extension-release.strict' extended attribute from file, ignoring: %m", + extension_release_dir_path, filename); + if (r > 0) { + log_debug("%s/%s: 'user.extension-release.strict' attribute is true, ignoring file.", + extension_release_dir_path, filename); + return true; + } + + log_debug("%s/%s: 'user.extension-release.strict' attribute is false%s", + extension_release_dir_path, filename, + special_glyph(SPECIAL_GLYPH_ELLIPSIS)); + + return false; +} + int open_extension_release(const char *root, const char *extension, char **ret_path, int *ret_fd) { _cleanup_free_ char *q = NULL; int r, fd; @@ -123,34 +161,12 @@ int open_extension_release(const char *root, const char *extension, char **ret_p continue; } - /* No xattr or cannot parse it? Then skip this. */ - _cleanup_free_ char *extension_release_xattr = NULL; - k = fgetxattr_malloc(extension_release_fd, "user.extension-release.strict", &extension_release_xattr); - if (k < 0 && !ERRNO_IS_XATTR_ABSENT(k)) - log_debug_errno(k, - "%s/%s: Failed to read 'user.extension-release.strict' extended attribute from file: %m", - extension_release_dir_path, de->d_name); - if (k < 0) { - log_debug("%s/%s does not have user.extension-release.strict xattr, ignoring.", extension_release_dir_path, de->d_name); - continue; - } - - /* Explicitly set to request strict matching? Skip it. */ - k = parse_boolean(extension_release_xattr); - if (k < 0) - log_debug_errno(k, - "%s/%s: Failed to parse 'user.extension-release.strict' extended attribute from file: %m", - extension_release_dir_path, de->d_name); - else if (k > 0) - log_debug("%s/%s: 'user.extension-release.strict' attribute is true, ignoring file.", - extension_release_dir_path, de->d_name); + k = extension_release_strict_xattr_value(extension_release_fd, + extension_release_dir_path, + de->d_name); if (k != 0) continue; - log_debug("%s/%s: 'user.extension-release.strict' attribute is false%s", - extension_release_dir_path, de->d_name, - special_glyph(SPECIAL_GLYPH_ELLIPSIS)); - /* We already found what we were looking for, but there's another candidate? * We treat this as an error, as we want to enforce that there are no ambiguities * in case we are in the fallback path.*/ |