Add send_broadcast as an attribute of <allow> and <deny> elements
authorSimon McVittie <smcv@collabora.com>
Wed, 19 Jul 2017 14:53:17 +0000 (15:53 +0100)
committerSimon McVittie <smcv@debian.org>
Fri, 28 Jul 2017 10:24:20 +0000 (11:24 +0100)
commit7424c3499f1cfafabfca49710e757eb43813f65e
treeb4ddb57785ada4a7ec4e4093c918964fcda9d9bf
parent2d3f751716c64c3d7997bbcd362df5e4a6b1d63e
Add send_broadcast as an attribute of <allow> and <deny> elements

<allow send_broadcast="true" ...> only matches broadcasts,
which are signals with a NULL destination. There was previously
no way for the policy language to express "NULL destination",
only "any destination".

<allow send_broadcast="false" ...> only matches non-broadcasts,
which are non-signals or signals with a non-NULL destination.
There was previously no way for the policy language to express
"any non-NULL destination", only "any destination".

Reviewed-by: Philip Withnall <withnall@endlessm.com>
[smcv: improved documentation as per Philip's review]
Signed-off-by: Simon McVittie <smcv@collabora.com>
Reviewed-by: Thiago Macieira <thiago@kde.org>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92853
bus/config-parser.c
bus/policy.c
bus/policy.h
doc/dbus-daemon.1.xml.in