summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon McVittie <simon.mcvittie@collabora.co.uk>2016-10-07 19:16:16 +0100
committerSimon McVittie <smcv@debian.org>2016-10-13 22:44:12 +0100
commit352ae6de6eac1927c9cb999b946df78bdfe22d1c (patch)
tree24a873ca66ae105cda8c47359579c8b38c53c0a9
parentb2f33623a0a9981c94a4260c3231500f06f264ea (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.c4
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;
}