chiark / gitweb /
bus-proxy: fix policy for expected/non-expected reply tags
authorDavid Herrmann <dh.herrmann@gmail.com>
Sun, 11 Jan 2015 13:13:19 +0000 (14:13 +0100)
committerDavid Herrmann <dh.herrmann@gmail.com>
Sun, 11 Jan 2015 13:18:29 +0000 (14:18 +0100)
commit0042d824e3616aaf2e3eec23d3b2e6aec7c0470c
treee9d121982239d984440c88a1f90aae6fc19327c4
parent24b759c5d79c1a4993c05c1ef7f44f9ff6d7f463
bus-proxy: fix policy for expected/non-expected reply tags

dbus-1 distinguishes expected and non-expected replies. An expected reply
is a reply that is sent as answer to a previously forwarded method-call
before the timeout fires. Those replies are, by default, forwarded and
DENY policy tags are ignored on them (unless explicitly stated otherwise).

We don't track reply-windows in the bus-proxy as the kernel already does
this. Furthermore, the kernel prohibits any non-expected replies (which
breaks dbus-1, but it was an odd feature, anyway).

Therefore, skip policy checks on replies and always let the kernel deal
with it!

To be correct, we should still process DENY tags marked as
send_expected_reply=true (which is *NOT* the default!). However, so far we
don't parse those attributes, and no-one really uses it, so lets not
implement it for now. It's marked as TODO if anyone feels like fixing it.
src/bus-proxyd/bus-proxyd.c