[PATCH] connman: don't give user 'system' permissions by default. We might not have one. (connman 0.46)

[PATCH] connman: don't give user 'system' permissions by default. We might not have one. (connman 0.46)

[Sebastian Spaeth]

Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
---
 recipes/connman/connman_0.46.bb                  |    1 +
 recipes/connman/files/no_system_user_perms.patch |   14 ++++++++++++++
 2 files changed, 15 insertions(+), 0 deletions(-)
 create mode 100644 recipes/connman/files/no_system_user_perms.patch

diff --git a/recipes/connman/connman_0.46.bb b/recipes/connman/connman_0.46.bb
index 26b7b82..b1729c6 100644
--- a/recipes/connman/connman_0.46.bb
+++ b/recipes/connman/connman_0.46.bb
@@ -32,6 +32,7 @@ EXTRA_OECONF += "\
 
 SRC_URI  = "\
   http://www.kernel.org/pub/linux/network/connman/connman-${PV}.tar.gz \
+  file://no_system_user_perms.patch;patch=1 \
   file://connman \
 "
 
diff --git a/recipes/connman/files/no_system_user_perms.patch b/recipes/connman/files/no_system_user_perms.patch
new file mode 100644
index 0000000..e02bd30
--- /dev/null
+++ b/recipes/connman/files/no_system_user_perms.patch
@@ -0,0 +1,14 @@
+diff --git a/src/connman-dbus.conf b/src/connman-dbus.conf
+index 2be3764..b4d4acd 100644
+--- a/src/connman-dbus.conf
++++ b/src/connman-dbus.conf
+@@ -6,9 +6,6 @@
+         <allow send_destination="org.moblin.connman"/>
+         <allow send_interface="org.moblin.connman.Agent"/>
+     </policy>
+-    <policy user="system">
+-        <allow send_destination="org.moblin.connman"/>
+-    </policy>
+     <policy at_console="true">
+         <allow send_destination="org.moblin.connman"/>
+     </policy>
-- 
1.6.3.3

Re: [PATCH] connman: don't give user 'system' permissions by default. We might not have one. (connman 0.46)

[Sebastian Spaeth]

[-- Attachment #1: Type: text/plain, Size: 204 bytes --]

Can I get an ACK for this? default connman configuration assumes that we
have a user called "system". This simply gets rid of that assumption.

I would push this in addition to a PR bump.

spaetz


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 268 bytes --]

Re: [PATCH] connman: don't give user 'system' permissions by default. We might not have one. (connman 0.46)

[Sebastian Spaeth]

[-- Attachment #1: Type: text/plain, Size: 82 bytes --]

Not having heard back anything negative,
I took the freedom to push this now.


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 268 bytes --]