Re: [Patch v3] sandbox: Use dbus-run-session instead of dbus-launch when available

2016-09-27 Thread Stephen Smalley
On 09/27/2016 09:49 AM, Laurent Bigonville wrote:
> Le 27/09/16 à 15:50, Stephen Smalley a écrit :
>> On 09/27/2016 08:39 AM, Laurent Bigonville wrote:
>>> From: Laurent Bigonville 
>>>
>>> According to dbus upstream: "dbus-launch is fairly horrible code,
>>> complicated by the historical need for it to support X11 autolaunching,
>>> so the D-Bus maintainers would like to move it out of the critical path
>>> and minimize its use."
>>>
>>> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=836289
>>>
>>> Signed-off-by: Laurent Bigonville 
>> Thanks, applied.
> Thanks,
> 
> Are there any plans to roll a new release of the userspace soon?

Yes, 2.6-rc1 is planned on 9/30, and then 2.6 will follow once it seems
stable.

___
Selinux mailing list
Selinux@tycho.nsa.gov
To unsubscribe, send email to selinux-le...@tycho.nsa.gov.
To get help, send an email containing "help" to selinux-requ...@tycho.nsa.gov.

Re: [Patch v3] sandbox: Use dbus-run-session instead of dbus-launch when available

2016-09-27 Thread Laurent Bigonville

Le 27/09/16 à 15:50, Stephen Smalley a écrit :

On 09/27/2016 08:39 AM, Laurent Bigonville wrote:

From: Laurent Bigonville 

According to dbus upstream: "dbus-launch is fairly horrible code,
complicated by the historical need for it to support X11 autolaunching,
so the D-Bus maintainers would like to move it out of the critical path
and minimize its use."

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=836289

Signed-off-by: Laurent Bigonville 

Thanks, applied.

Thanks,

Are there any plans to roll a new release of the userspace soon?
___
Selinux mailing list
Selinux@tycho.nsa.gov
To unsubscribe, send email to selinux-le...@tycho.nsa.gov.
To get help, send an email containing "help" to selinux-requ...@tycho.nsa.gov.

[Patch v3] sandbox: Use dbus-run-session instead of dbus-launch when available

2016-09-27 Thread Laurent Bigonville
From: Laurent Bigonville 

According to dbus upstream: "dbus-launch is fairly horrible code,
complicated by the historical need for it to support X11 autolaunching,
so the D-Bus maintainers would like to move it out of the critical path
and minimize its use."

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=836289

Signed-off-by: Laurent Bigonville 
---
 policycoreutils/sandbox/sandbox | 8 ++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/policycoreutils/sandbox/sandbox b/policycoreutils/sandbox/sandbox
index b6bc805..4010e83 100644
--- a/policycoreutils/sandbox/sandbox
+++ b/policycoreutils/sandbox/sandbox
@@ -285,9 +285,13 @@ class Sandbox:
 # /usr/bin/test -r ~/.xmodmap && /usr/bin/xmodmap ~/.xmodmap
 %s &
 WM_PID=$!
-dbus-launch --exit-with-session %s
+if which dbus-run-session >/dev/null 2>&1; then
+dbus-run-session -- %s
+else
+dbus-launch --exit-with-session %s
+fi
 kill -TERM $WM_PID  2> /dev/null
-""" % (command, wm, command))
+""" % (command, wm, command, command))
 fd.close()
 os.chmod(execfile, 0o700)
 
-- 
2.9.3

___
Selinux mailing list
Selinux@tycho.nsa.gov
To unsubscribe, send email to selinux-le...@tycho.nsa.gov.
To get help, send an email containing "help" to selinux-requ...@tycho.nsa.gov.