From 38e8a6c7fdffd3389cb3596139b1309579193946 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Thu, 17 Mar 2022 10:16:30 +0100 Subject: [PATCH] shared/install: stop passing duplicate root argument to install_name_printf() All callers were just passing info + info->root, we can simplify this. --- src/shared/install-printf.c | 7 +++---- src/shared/install-printf.h | 3 +-- src/shared/install.c | 8 ++++---- src/test/test-load-fragment.c | 2 +- 4 files changed, 9 insertions(+), 11 deletions(-) diff --git a/src/shared/install-printf.c b/src/shared/install-printf.c index 571c4818e2..4c0eb3ce61 100644 --- a/src/shared/install-printf.c +++ b/src/shared/install-printf.c @@ -99,9 +99,8 @@ static int specifier_last_component(char specifier, const void *data, const char int install_name_printf( UnitFileScope scope, - const UnitFileInstallInfo *i, + const UnitFileInstallInfo *info, const char *format, - const char *root, char **ret) { /* This is similar to unit_name_printf() */ @@ -118,9 +117,9 @@ int install_name_printf( {} }; - assert(i); + assert(info); assert(format); assert(ret); - return specifier_printf(format, UNIT_NAME_MAX, table, root, i, ret); + return specifier_printf(format, UNIT_NAME_MAX, table, info->root, info, ret); } diff --git a/src/shared/install-printf.h b/src/shared/install-printf.h index d2cccdf66d..60d3a9fc55 100644 --- a/src/shared/install-printf.h +++ b/src/shared/install-printf.h @@ -6,7 +6,6 @@ int install_name_printf( UnitFileScope scope, - const UnitFileInstallInfo *i, + const UnitFileInstallInfo *info, const char *format, - const char *root, char **ret); diff --git a/src/shared/install.c b/src/shared/install.c index 91e38c57cc..d5e1979280 100644 --- a/src/shared/install.c +++ b/src/shared/install.c @@ -1203,7 +1203,7 @@ static int config_parse_also( if (r == 0) break; - r = install_name_printf(ctx->scope, info, word, info->root, &printed); + r = install_name_printf(ctx->scope, info, word, &printed); if (r < 0) return log_syntax(unit, LOG_WARNING, filename, line, r, "Failed to resolve unit name in Also=\"%s\": %m", word); @@ -1252,7 +1252,7 @@ static int config_parse_default_instance( return log_syntax(unit, LOG_WARNING, filename, line, 0, "DefaultInstance= only makes sense for template units, ignoring."); - r = install_name_printf(ctx->scope, info, rvalue, info->root, &printed); + r = install_name_printf(ctx->scope, info, rvalue, &printed); if (r < 0) return log_syntax(unit, LOG_WARNING, filename, line, r, "Failed to resolve instance name in DefaultInstance=\"%s\": %m", rvalue); @@ -1845,7 +1845,7 @@ static int install_info_symlink_alias( STRV_FOREACH(s, info->aliases) { _cleanup_free_ char *alias_path = NULL, *dst = NULL, *dst_updated = NULL; - q = install_name_printf(scope, info, *s, info->root, &dst); + q = install_name_printf(scope, info, *s, &dst); if (q < 0) { unit_file_changes_add(changes, n_changes, q, *s, NULL); r = r < 0 ? r : q; @@ -1935,7 +1935,7 @@ static int install_info_symlink_wants( STRV_FOREACH(s, list) { _cleanup_free_ char *dst = NULL; - q = install_name_printf(scope, info, *s, info->root, &dst); + q = install_name_printf(scope, info, *s, &dst); if (q < 0) { unit_file_changes_add(changes, n_changes, q, *s, NULL); return q; diff --git a/src/test/test-load-fragment.c b/src/test/test-load-fragment.c index 40cd7ea7e6..52341e5199 100644 --- a/src/test/test-load-fragment.c +++ b/src/test/test-load-fragment.c @@ -515,7 +515,7 @@ TEST(install_printf, .sd_booted = true) { _cleanup_free_ char *t = NULL, \ *d1 = ASSERT_PTR(strdup(i.name)), \ *d2 = ASSERT_PTR(strdup(i.path)); \ - int r = install_name_printf(scope, &src, pattern, NULL, &t); \ + int r = install_name_printf(scope, &src, pattern, &t); \ assert_se(result ? r >= 0 : r < 0); \ memzero(i.name, strlen(i.name)); \ memzero(i.path, strlen(i.path)); \