Patchwork [v2] dbus: change permissions of dbus-daemon-launch-helper to 4755

login
register
mail settings
Submitter Steve Sakoman
Date Jan. 7, 2012, 4:51 a.m.
Message ID <1325911915-16001-2-git-send-email-steve@sakoman.com>
Download mbox | patch
Permalink /patch/18755/
State Accepted
Commit 7b1e227bf28c9d4f3bb16af4d9ac21265e88a3f1
Headers show

Comments

Steve Sakoman - Jan. 7, 2012, 4:51 a.m.
With current permissions of 4754 on systemd systems various services will fail to
activate with "helpful" error messages of the type:

2000-01-01T00:00:25+00:00 omap3-multi dbus[178]: [system] Activated service
'org.freedesktop.nm_dispatcher' failed: Failed to execute program
/usr/libexec/dbus-daemon-launch-helper: Success

Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/recipes-core/dbus/dbus.inc |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)
Saul Wold - Jan. 12, 2012, 6:27 a.m.
On 01/06/2012 08:51 PM, Steve Sakoman wrote:
> With current permissions of 4754 on systemd systems various services will fail to
> activate with "helpful" error messages of the type:
>
> 2000-01-01T00:00:25+00:00 omap3-multi dbus[178]: [system] Activated service
> 'org.freedesktop.nm_dispatcher' failed: Failed to execute program
> /usr/libexec/dbus-daemon-launch-helper: Success
>
> Signed-off-by: Steve Sakoman<steve@sakoman.com>
> ---
>   meta/recipes-core/dbus/dbus.inc |    4 +++-
>   1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
> index 99b2fc7..2a25256 100644
> --- a/meta/recipes-core/dbus/dbus.inc
> +++ b/meta/recipes-core/dbus/dbus.inc
> @@ -10,6 +10,8 @@ DEPENDS = "expat virtual/libintl ${@base_contains('DISTRO_FEATURES', 'x11', '${X
>   DEPENDS_virtclass-native = "expat-native virtual/libintl-native"
>   DEPENDS_virtclass-nativesdk = "expat-nativesdk virtual/libintl-nativesdk virtual/libx11"
>
> +INC_PR = "r1"
> +
>   SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \
>              file://tmpdir.patch; \
>              file://dbus-1.init"
> @@ -84,7 +86,7 @@ do_install() {
>   	chown messagebus:messagebus ${D}${localstatedir}/run/dbus ${D}${localstatedir}/lib/dbus
>
>   	chown root:messagebus ${D}${libexecdir}/dbus-daemon-launch-helper
> -	chmod 4754 ${D}${libexecdir}/dbus-daemon-launch-helper
> +	chmod 4755 ${D}${libexecdir}/dbus-daemon-launch-helper
>
>   	# Remove Red Hat initscript
>   	rm -rf ${D}${sysconfdir}/rc.d

Merged into OE-Core

Thanks
	Sau!

Patch

diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc
index 99b2fc7..2a25256 100644
--- a/meta/recipes-core/dbus/dbus.inc
+++ b/meta/recipes-core/dbus/dbus.inc
@@ -10,6 +10,8 @@  DEPENDS = "expat virtual/libintl ${@base_contains('DISTRO_FEATURES', 'x11', '${X
 DEPENDS_virtclass-native = "expat-native virtual/libintl-native"
 DEPENDS_virtclass-nativesdk = "expat-nativesdk virtual/libintl-nativesdk virtual/libx11"
 
+INC_PR = "r1"
+
 SRC_URI = "http://dbus.freedesktop.org/releases/dbus/dbus-${PV}.tar.gz \
            file://tmpdir.patch; \
            file://dbus-1.init"
@@ -84,7 +86,7 @@  do_install() {
 	chown messagebus:messagebus ${D}${localstatedir}/run/dbus ${D}${localstatedir}/lib/dbus
 
 	chown root:messagebus ${D}${libexecdir}/dbus-daemon-launch-helper
-	chmod 4754 ${D}${libexecdir}/dbus-daemon-launch-helper
+	chmod 4755 ${D}${libexecdir}/dbus-daemon-launch-helper
 
 	# Remove Red Hat initscript
 	rm -rf ${D}${sysconfdir}/rc.d