mirror of
https://github.com/systemd/systemd-stable.git
synced 2024-10-27 18:55:09 +03:00
sd-bus: verify destination and sender values when setting
We would verify destination e.g. in sd_bus_message_new_call, but allow setting any value later on with sd_bus_message_set_destination. I assume this check was omitted not on purpose.
This commit is contained in:
parent
9c9207912e
commit
3d51a011f1
@ -5433,6 +5433,7 @@ int bus_message_parse_fields(sd_bus_message *m) {
|
||||
_public_ int sd_bus_message_set_destination(sd_bus_message *m, const char *destination) {
|
||||
assert_return(m, -EINVAL);
|
||||
assert_return(destination, -EINVAL);
|
||||
assert_return(service_name_is_valid(destination), -EINVAL);
|
||||
assert_return(!m->sealed, -EPERM);
|
||||
assert_return(!m->destination, -EEXIST);
|
||||
|
||||
@ -5442,6 +5443,7 @@ _public_ int sd_bus_message_set_destination(sd_bus_message *m, const char *desti
|
||||
_public_ int sd_bus_message_set_sender(sd_bus_message *m, const char *sender) {
|
||||
assert_return(m, -EINVAL);
|
||||
assert_return(sender, -EINVAL);
|
||||
assert_return(service_name_is_valid(sender), -EINVAL);
|
||||
assert_return(!m->sealed, -EPERM);
|
||||
assert_return(!m->sender, -EEXIST);
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user