ryantimwilson / rpms / systemd

Forked from rpms/systemd a month ago
Clone
Zbigniew Jędrzejewski-Szmek b3e1d5
From f62d1f6ea55fc0dcccbe60582804c9b033f8ad0e Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek b3e1d5
From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= <zbyszek@in.waw.pl>
Zbigniew Jędrzejewski-Szmek b3e1d5
Date: Tue, 18 Jun 2024 20:32:10 +0200
Zbigniew Jędrzejewski-Szmek b3e1d5
Subject: [PATCH] tmpfiles: make --purge hard to (mis-)use
Zbigniew Jędrzejewski-Szmek b3e1d5
Zbigniew Jędrzejewski-Szmek b3e1d5
Follow-up for https://github.com/systemd/systemd/pull/33383.
Zbigniew Jędrzejewski-Szmek b3e1d5
---
Zbigniew Jędrzejewski-Szmek b3e1d5
 src/tmpfiles/tmpfiles.c | 17 +++++++++++++++++
Zbigniew Jędrzejewski-Szmek b3e1d5
 1 file changed, 17 insertions(+)
Zbigniew Jędrzejewski-Szmek b3e1d5
Zbigniew Jędrzejewski-Szmek b3e1d5
diff --git a/src/tmpfiles/tmpfiles.c b/src/tmpfiles/tmpfiles.c
Zbigniew Jędrzejewski-Szmek b3e1d5
index 5841db293e..9b0f744ba9 100644
Zbigniew Jędrzejewski-Szmek b3e1d5
--- a/src/tmpfiles/tmpfiles.c
Zbigniew Jędrzejewski-Szmek b3e1d5
+++ b/src/tmpfiles/tmpfiles.c
Zbigniew Jędrzejewski-Szmek b3e1d5
@@ -4195,6 +4195,7 @@ static int parse_argv(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek b3e1d5
                 ARG_IMAGE_POLICY,
Zbigniew Jędrzejewski-Szmek b3e1d5
                 ARG_REPLACE,
Zbigniew Jędrzejewski-Szmek b3e1d5
                 ARG_DRY_RUN,
Zbigniew Jędrzejewski-Szmek b3e1d5
+                ARG_DESTROY_DATA,
Zbigniew Jędrzejewski-Szmek b3e1d5
                 ARG_NO_PAGER,
Zbigniew Jędrzejewski-Szmek b3e1d5
         };
Zbigniew Jędrzejewski-Szmek b3e1d5
 
Zbigniew Jędrzejewski-Szmek b3e1d5
@@ -4218,10 +4219,18 @@ static int parse_argv(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek b3e1d5
                 { "replace",        required_argument,   NULL, ARG_REPLACE        },
Zbigniew Jędrzejewski-Szmek b3e1d5
                 { "dry-run",        no_argument,         NULL, ARG_DRY_RUN        },
Zbigniew Jędrzejewski-Szmek b3e1d5
                 { "no-pager",       no_argument,         NULL, ARG_NO_PAGER       },
Zbigniew Jędrzejewski-Szmek b3e1d5
+
Zbigniew Jędrzejewski-Szmek b3e1d5
+                /* This is not documented on purpose.
Zbigniew Jędrzejewski-Szmek b3e1d5
+                 * If you think --purge should be allowed without jumping through hoops,
Zbigniew Jędrzejewski-Szmek b3e1d5
+                 * consider opening a bug report with the description of the use case.
Zbigniew Jędrzejewski-Szmek b3e1d5
+                 */
Zbigniew Jędrzejewski-Szmek b3e1d5
+                { "destroy-data",   no_argument,         NULL, ARG_DESTROY_DATA   },
Zbigniew Jędrzejewski-Szmek b3e1d5
+
Zbigniew Jędrzejewski-Szmek b3e1d5
                 {}
Zbigniew Jędrzejewski-Szmek b3e1d5
         };
Zbigniew Jędrzejewski-Szmek b3e1d5
 
Zbigniew Jędrzejewski-Szmek b3e1d5
         int c, r;
Zbigniew Jędrzejewski-Szmek b3e1d5
+        bool destroy_data = false;
Zbigniew Jędrzejewski-Szmek b3e1d5
 
Zbigniew Jędrzejewski-Szmek b3e1d5
         assert(argc >= 0);
Zbigniew Jędrzejewski-Szmek b3e1d5
         assert(argv);
Zbigniew Jędrzejewski-Szmek b3e1d5
@@ -4328,6 +4337,10 @@ static int parse_argv(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek b3e1d5
                         arg_dry_run = true;
Zbigniew Jędrzejewski-Szmek b3e1d5
                         break;
Zbigniew Jędrzejewski-Szmek b3e1d5
 
Zbigniew Jędrzejewski-Szmek b3e1d5
+                case ARG_DESTROY_DATA:
Zbigniew Jędrzejewski-Szmek b3e1d5
+                        destroy_data = true;
Zbigniew Jędrzejewski-Szmek b3e1d5
+                        break;
Zbigniew Jędrzejewski-Szmek b3e1d5
+
Zbigniew Jędrzejewski-Szmek b3e1d5
                 case ARG_NO_PAGER:
Zbigniew Jędrzejewski-Szmek b3e1d5
                         arg_pager_flags |= PAGER_DISABLE;
Zbigniew Jędrzejewski-Szmek b3e1d5
                         break;
Zbigniew Jędrzejewski-Szmek b3e1d5
@@ -4347,6 +4360,10 @@ static int parse_argv(int argc, char *argv[]) {
Zbigniew Jędrzejewski-Szmek b3e1d5
                 return log_error_errno(SYNTHETIC_ERRNO(EINVAL),
Zbigniew Jędrzejewski-Szmek b3e1d5
                                        "Refusing --purge without specification of a configuration file.");
Zbigniew Jędrzejewski-Szmek b3e1d5
 
Zbigniew Jędrzejewski-Szmek b3e1d5
+        if (FLAGS_SET(arg_operation, OPERATION_PURGE) && !arg_dry_run && !destroy_data)
Zbigniew Jędrzejewski-Szmek b3e1d5
+                return log_error_errno(SYNTHETIC_ERRNO(EINVAL),
Zbigniew Jędrzejewski-Szmek b3e1d5
+                                       "Refusing --purge without --destroy-data.");
Zbigniew Jędrzejewski-Szmek b3e1d5
+
Zbigniew Jędrzejewski-Szmek b3e1d5
         if (arg_replace && arg_cat_flags != CAT_CONFIG_OFF)
Zbigniew Jędrzejewski-Szmek b3e1d5
                 return log_error_errno(SYNTHETIC_ERRNO(EINVAL),
Zbigniew Jędrzejewski-Szmek b3e1d5
                                        "Option --replace= is not supported with --cat-config/--tldr.");