Patchwork [Consolidated,Pull,17/22] connman: Document the ACL-breaking sed

login
mail settings
Submitter Elizabeth Flanagan
Date June 7, 2012, 11:37 p.m.
Message ID <776c4b947a27f11117fd9ce412627c26f54efef8.1339111944.git.elizabeth.flanagan@intel.com>
Download mbox | patch
Permalink /patch/29431/
State New
Headers show

Comments

Elizabeth Flanagan - June 7, 2012, 11:37 p.m.
From: Ross Burton <ross.burton@intel.com>

Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Elizabeth Flanagan <elizabeth.flanagan@intel.com>
---
 meta/recipes-connectivity/connman/connman.inc |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

Patch

diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
index 58e3f56..4fb7b43 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -53,6 +53,9 @@  USERADD_PARAM_${PN} = "--system --no-create-home \
 
 inherit autotools pkgconfig update-rc.d useradd
 
+# This allows *everyone* to access ConnMan over DBus, without any access
+# control.  Really the at_console flag should work, which would mean that
+# both this and the xuser patch can be dropped.
 do_compile_append() {
 	sed -i -e s:deny:allow:g src/connman-dbus.conf
 }