sysusers: add --dry-run

This commit is contained in:
Zbigniew Jędrzejewski-Szmek 2021-12-08 10:33:53 +01:00
parent 98167876a0
commit 64fe10957b
2 changed files with 40 additions and 4 deletions

View File

@ -115,6 +115,12 @@
</listitem> </listitem>
</varlistentry> </varlistentry>
<varlistentry>
<term><option>--dry-run</option></term>
<listitem><para>Process the configuration and figure out what entries would be created, but don't
actually write anything.</para></listitem>
</varlistentry>
<varlistentry> <varlistentry>
<term><option>--inline</option></term> <term><option>--inline</option></term>
<listitem><para>Treat each positional argument as a separate configuration <listitem><para>Treat each positional argument as a separate configuration

View File

@ -75,6 +75,7 @@ static char *arg_root = NULL;
static char *arg_image = NULL; static char *arg_image = NULL;
static bool arg_cat_config = false; static bool arg_cat_config = false;
static const char *arg_replace = NULL; static const char *arg_replace = NULL;
static bool arg_dry_run = false;
static bool arg_inline = false; static bool arg_inline = false;
static PagerFlags arg_pager_flags = 0; static PagerFlags arg_pager_flags = 0;
@ -390,6 +391,11 @@ static int write_temporary_passwd(const char *passwd_path, FILE **tmpfile, char
if (ordered_hashmap_isempty(todo_uids)) if (ordered_hashmap_isempty(todo_uids))
return 0; return 0;
if (arg_dry_run) {
log_info("Would write /etc/passwd…");
return 0;
}
r = fopen_temporary_label("/etc/passwd", passwd_path, &passwd, &passwd_tmp); r = fopen_temporary_label("/etc/passwd", passwd_path, &passwd, &passwd_tmp);
if (r < 0) if (r < 0)
return log_debug_errno(r, "Failed to open temporary copy of %s: %m", passwd_path); return log_debug_errno(r, "Failed to open temporary copy of %s: %m", passwd_path);
@ -509,6 +515,11 @@ static int write_temporary_shadow(const char *shadow_path, FILE **tmpfile, char
if (ordered_hashmap_isempty(todo_uids)) if (ordered_hashmap_isempty(todo_uids))
return 0; return 0;
if (arg_dry_run) {
log_info("Would write /etc/shadow…");
return 0;
}
r = fopen_temporary_label("/etc/shadow", shadow_path, &shadow, &shadow_tmp); r = fopen_temporary_label("/etc/shadow", shadow_path, &shadow, &shadow_tmp);
if (r < 0) if (r < 0)
return log_debug_errno(r, "Failed to open temporary copy of %s: %m", shadow_path); return log_debug_errno(r, "Failed to open temporary copy of %s: %m", shadow_path);
@ -642,6 +653,11 @@ static int write_temporary_group(const char *group_path, FILE **tmpfile, char **
if (ordered_hashmap_isempty(todo_gids) && ordered_hashmap_isempty(members)) if (ordered_hashmap_isempty(todo_gids) && ordered_hashmap_isempty(members))
return 0; return 0;
if (arg_dry_run) {
log_info("Would write /etc/group…");
return 0;
}
r = fopen_temporary_label("/etc/group", group_path, &group, &group_tmp); r = fopen_temporary_label("/etc/group", group_path, &group, &group_tmp);
if (r < 0) if (r < 0)
return log_debug_errno(r, "Failed to open temporary copy of %s: %m", group_path); return log_debug_errno(r, "Failed to open temporary copy of %s: %m", group_path);
@ -743,6 +759,11 @@ static int write_temporary_gshadow(const char * gshadow_path, FILE **tmpfile, ch
if (ordered_hashmap_isempty(todo_gids) && ordered_hashmap_isempty(members)) if (ordered_hashmap_isempty(todo_gids) && ordered_hashmap_isempty(members))
return 0; return 0;
if (arg_dry_run) {
log_info("Would write /etc/gshadow…");
return 0;
}
r = fopen_temporary_label("/etc/gshadow", gshadow_path, &gshadow, &gshadow_tmp); r = fopen_temporary_label("/etc/gshadow", gshadow_path, &gshadow, &gshadow_tmp);
if (r < 0) if (r < 0)
return log_debug_errno(r, "Failed to open temporary copy of %s: %m", gshadow_path); return log_debug_errno(r, "Failed to open temporary copy of %s: %m", gshadow_path);
@ -1825,6 +1846,7 @@ static int help(void) {
" --root=PATH Operate on an alternate filesystem root\n" " --root=PATH Operate on an alternate filesystem root\n"
" --image=PATH Operate on disk image as filesystem root\n" " --image=PATH Operate on disk image as filesystem root\n"
" --replace=PATH Treat arguments as replacement for PATH\n" " --replace=PATH Treat arguments as replacement for PATH\n"
" --dry-run Just print what would be done\n"
" --inline Treat arguments as configuration lines\n" " --inline Treat arguments as configuration lines\n"
" --no-pager Do not pipe output into a pager\n" " --no-pager Do not pipe output into a pager\n"
"\nSee the %s for details.\n", "\nSee the %s for details.\n",
@ -1842,6 +1864,7 @@ static int parse_argv(int argc, char *argv[]) {
ARG_ROOT, ARG_ROOT,
ARG_IMAGE, ARG_IMAGE,
ARG_REPLACE, ARG_REPLACE,
ARG_DRY_RUN,
ARG_INLINE, ARG_INLINE,
ARG_NO_PAGER, ARG_NO_PAGER,
}; };
@ -1851,8 +1874,9 @@ static int parse_argv(int argc, char *argv[]) {
{ "version", no_argument, NULL, ARG_VERSION }, { "version", no_argument, NULL, ARG_VERSION },
{ "cat-config", no_argument, NULL, ARG_CAT_CONFIG }, { "cat-config", no_argument, NULL, ARG_CAT_CONFIG },
{ "root", required_argument, NULL, ARG_ROOT }, { "root", required_argument, NULL, ARG_ROOT },
{ "image", required_argument, NULL, ARG_IMAGE }, { "image", required_argument, NULL, ARG_IMAGE },
{ "replace", required_argument, NULL, ARG_REPLACE }, { "replace", required_argument, NULL, ARG_REPLACE },
{ "dry-run", no_argument, NULL, ARG_DRY_RUN },
{ "inline", no_argument, NULL, ARG_INLINE }, { "inline", no_argument, NULL, ARG_INLINE },
{ "no-pager", no_argument, NULL, ARG_NO_PAGER }, { "no-pager", no_argument, NULL, ARG_NO_PAGER },
{} {}
@ -1903,6 +1927,10 @@ static int parse_argv(int argc, char *argv[]) {
arg_replace = optarg; arg_replace = optarg;
break; break;
case ARG_DRY_RUN:
arg_dry_run = true;
break;
case ARG_INLINE: case ARG_INLINE:
arg_inline = true; arg_inline = true;
break; break;
@ -2077,9 +2105,11 @@ static int run(int argc, char *argv[]) {
if (r < 0) if (r < 0)
return r; return r;
lock = take_etc_passwd_lock(arg_root); if (!arg_dry_run) {
if (lock < 0) lock = take_etc_passwd_lock(arg_root);
return log_error_errno(lock, "Failed to take /etc/passwd lock: %m"); if (lock < 0)
return log_error_errno(lock, "Failed to take /etc/passwd lock: %m");
}
r = load_user_database(); r = load_user_database();
if (r < 0) if (r < 0)