transaction: drop bus error arg from transaction_add_propagate_reload_jobs()

We erase the error anyway always, hence no point in even passing
anything in from callers.

diff --git a/src/core/manager.c b/src/core/manager.c
index 34a6d7db2cd64e0008f6e764cd70170ecf21bf31..23df5ce1913e9f9ab30ec8040d742f8314905fa3 100644 (file)
--- a/src/core/manager.c
+++ b/src/core/manager.c
@@ -2147,8 +2147,7 @@ int manager_propagate_reload(Manager *m, Unit *unit, JobMode mode, sd_bus_error
                         tr,
                         unit,
                         tr->anchor_job,
-                        mode == JOB_IGNORE_DEPENDENCIES ? TRANSACTION_IGNORE_ORDER : 0,
-                        e);
+                        mode == JOB_IGNORE_DEPENDENCIES ? TRANSACTION_IGNORE_ORDER : 0);
 
         r = transaction_activate(tr, m, mode, NULL, e);
         if (r < 0)

diff --git a/src/core/transaction.c b/src/core/transaction.c
index dda17c6c16bb6a2893a75644218175203cd916de..9c1523c47ad2824357a64ae1ccd70f231d27ad2b 100644 (file)
--- a/src/core/transaction.c
+++ b/src/core/transaction.c
@@ -878,8 +878,7 @@ void transaction_add_propagate_reload_jobs(
                 Transaction *tr,
                 Unit *unit,
                 Job *by,
-                TransactionAddFlags flags,
-                sd_bus_error *e) {
+                TransactionAddFlags flags) {
 
         JobType nt;
         Unit *dep;
@@ -889,17 +888,17 @@ void transaction_add_propagate_reload_jobs(
         assert(unit);
 
         UNIT_FOREACH_DEPENDENCY(dep, unit, UNIT_ATOM_PROPAGATES_RELOAD_TO) {
+                _cleanup_(sd_bus_error_free) sd_bus_error e = SD_BUS_ERROR_NULL;
+
                 nt = job_type_collapse(JOB_TRY_RELOAD, dep);
                 if (nt == JOB_NOP)
                         continue;
 
-                r = transaction_add_job_and_dependencies(tr, nt, dep, by, flags, e);
-                if (r < 0) {
+                r = transaction_add_job_and_dependencies(tr, nt, dep, by, flags, &e);
+                if (r < 0)
                         log_unit_warning(dep,
                                          "Cannot add dependency reload job, ignoring: %s",
-                                         bus_error_message(e, r));
-                        sd_bus_error_free(e);
-                }
+                                         bus_error_message(&e, r));
         }
 }
 
@@ -1099,7 +1098,7 @@ int transaction_add_job_and_dependencies(
                 }
 
                 if (type == JOB_RELOAD)
-                        transaction_add_propagate_reload_jobs(tr, ret->unit, ret, flags & TRANSACTION_IGNORE_ORDER, e);
+                        transaction_add_propagate_reload_jobs(tr, ret->unit, ret, flags & TRANSACTION_IGNORE_ORDER);
 
                 /* JOB_VERIFY_ACTIVE requires no dependency handling */
         }

diff --git a/src/core/transaction.h b/src/core/transaction.h
index 6ee0872c2738fb5bed04689a698cf3085d90622c..5874077aefdc4b3981a434101859190a7249dd69 100644 (file)
--- a/src/core/transaction.h
+++ b/src/core/transaction.h
@@ -30,8 +30,7 @@ typedef enum TransactionAddFlags {
 void transaction_add_propagate_reload_jobs(
                 Transaction *tr,
                 Unit *unit, Job *by,
-                TransactionAddFlags flags,
-                sd_bus_error *e);
+                TransactionAddFlags flags);
 
 int transaction_add_job_and_dependencies(
                 Transaction *tr,