diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2016-10-07 19:16:16 +0100 |
|---|---|---|
| committer | Simon McVittie <smcv@debian.org> | 2016-10-13 22:44:12 +0100 |
| commit | 352ae6de6eac1927c9cb999b946df78bdfe22d1c (patch) | |
| tree | 24a873ca66ae105cda8c47359579c8b38c53c0a9 | |
| parent | b2f33623a0a9981c94a4260c3231500f06f264ea (diff) | |
Print errors parsing match rules correctly
Not an exploitable vulnerability, just incorrect output.
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
| -rw-r--r-- | bus/signals.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/bus/signals.c b/bus/signals.c index f28aaa3a..838161f7 100644 --- a/bus/signals.c +++ b/bus/signals.c @@ -871,7 +871,7 @@ bus_match_rule_parse_arg_match (BusMatchRule *rule, else { dbus_set_error (error, DBUS_ERROR_MATCH_RULE_INVALID, - "Key '%s' in match rule contains junk after argument number (%u). Only 'arg%upath' (for example) or 'arg0namespace' are valid", key, arg, arg); + "Key '%s' in match rule contains junk after argument number (%lu). Only 'arg%lupath' (for example) or 'arg0namespace' are valid", key, arg, arg); goto failed; } } @@ -889,7 +889,7 @@ bus_match_rule_parse_arg_match (BusMatchRule *rule, rule->args[arg] != NULL) { dbus_set_error (error, DBUS_ERROR_MATCH_RULE_INVALID, - "Argument %d matched more than once in match rule\n", key); + "Argument %s matched more than once in match rule\n", key); goto failed; } |
