diff options
| author | Lincoln Ramsay <a1291762@gmail.com> | 2021-03-24 17:37:25 +1000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-24 16:37:25 +0900 |
| commit | 7325a2b2d15af09a9389723d6153050130c0bd36 (patch) | |
| tree | 6ee70506c074827ba2d545e9310cecdc334b7194 | |
| parent | 6b1ed5e7e68fc5992a7bdabe4a05a7a3e1e1d898 (diff) | |
| download | systemd-7325a2b2d15af09a9389723d6153050130c0bd36.tar.gz | |
pid1: do not use generated strings as format strings (#19098)
The generated string may include %, which will confuse both the
xprintf call, and the VA_FORMAT_ADVANCE macro.
Pass the generated string as an argument to a "%s" format string
instead.
| -rw-r--r-- | src/core/transaction.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/core/transaction.c b/src/core/transaction.c index d7388e2367..1d3cf8f1fc 100644 --- a/src/core/transaction.c +++ b/src/core/transaction.c @@ -405,7 +405,7 @@ static int transaction_verify_order_one(Transaction *tr, Job *j, Job *from, unsi j->unit->id, unit_id == array ? "ordering cycle" : "dependency", *unit_id, *job_type, - unit_ids); + "%s", unit_ids); if (delete) { const char *status; @@ -414,7 +414,7 @@ static int transaction_verify_order_one(Transaction *tr, Job *j, Job *from, unsi "MESSAGE=%s: Job %s/%s deleted to break ordering cycle starting with %s/%s", j->unit->id, delete->unit->id, job_type_to_string(delete->type), j->unit->id, job_type_to_string(j->type), - unit_ids); + "%s", unit_ids); if (log_get_show_color()) status = ANSI_HIGHLIGHT_RED " SKIP " ANSI_NORMAL; @@ -432,7 +432,7 @@ static int transaction_verify_order_one(Transaction *tr, Job *j, Job *from, unsi log_struct(LOG_ERR, "MESSAGE=%s: Unable to break cycle starting with %s/%s", j->unit->id, j->unit->id, job_type_to_string(j->type), - unit_ids); + "%s", unit_ids); return sd_bus_error_setf(e, BUS_ERROR_TRANSACTION_ORDER_IS_CYCLIC, "Transaction order is cyclic. See system logs for details."); |