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

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

[Steve Sakoman]

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
-- 
1.7.1

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

[Saul Wold]

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!