Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-12 Thread Helge Kreutzmann
Hello Andreas Henriksson, On Sun, Aug 12, 2018 at 05:35:35PM +0200, Andreas Henriksson wrote: > Hello Helge Kreutzmann, > > Sorry if my comments sounded too negative, some more reasoning below. No problem. > On Sun, Aug 12, 2018 at 04:35:47PM +0200, Helge Kreutzmann wrote: > > Hello Andreas

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-12 Thread Andreas Henriksson
Hello Helge Kreutzmann, Sorry if my comments sounded too negative, some more reasoning below. On Sun, Aug 12, 2018 at 04:35:47PM +0200, Helge Kreutzmann wrote: > Hello Andreas Henriksson, > I'm a bit puzzled by your e-mail. Simon asked me to provide some text, > Chris prodded me and Davide and

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-12 Thread Helge Kreutzmann
Hello Andreas Henriksson, I'm a bit puzzled by your e-mail. Simon asked me to provide some text, Chris prodded me and Davide and Simon reviewed my text (which does not imply that it is perfect or so). On Sun, Aug 12, 2018 at 05:56:14AM +0200, Andreas Henriksson wrote: > Hello Helge Kreutzmann, >

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-11 Thread Andreas Henriksson
Hello Helge Kreutzmann, I'm skeptical how relevant it is to document how X and ssh works in the _su_ manpage, but either way since you're patching the manpage this is something you want to send upstream for review if you think it's worth persuing and my opinion doesn't count there. As always to

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-11 Thread Helge Kreutzmann
Hello Davide, hello Simon, On Thu, Aug 09, 2018 at 09:53:48PM +0200, Davide Prina wrote: > On 09/08/2018 21:06, Helge Kreutzmann wrote: > > >+Use a separate X display (e.g. "Switch User" in GNOME, or the equivalent > >fast-user-switching feature in other desktop environments), > > here probably

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-10 Thread Simon McVittie
On Thu, 09 Aug 2018 at 21:06:37 +0200, Helge Kreutzmann wrote: > +Further by default > +.B su > +does not allow the commands to access the current X display. This is perhaps misleading: su isn't allowing or denying anything, it's the X server that isn't allowing other users to access it. Perhaps

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-09 Thread Davide Prina
On 09/08/2018 21:06, Helge Kreutzmann wrote: +Use a separate X display (e.g. "Switch User" in GNOME, or the equivalent fast-user-switching feature in other desktop environments), here probably it is better to say that the user can switch from one to other user with the Ctrl-Alt-Fx keys

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-09 Thread Helge Kreutzmann
Hello Chris, On Wed, Aug 08, 2018 at 08:58:24PM +0200, Chris Hofstaedtler wrote: > * Helge Kreutzmann [180808 18:57]: > > On Tue, Aug 07, 2018 at 08:20:23PM +0100, Simon McVittie wrote: > > > Andreas already asked for a merge request, so it seems that proposing a > > > patch would indeed be

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-08 Thread Chris Hofstaedtler
* Helge Kreutzmann [180808 18:57]: > On Tue, Aug 07, 2018 at 08:20:23PM +0100, Simon McVittie wrote: > > Andreas already asked for a merge request, so it seems that proposing a > > patch would indeed be welcome. > > I'll do, incorporating your excellent explaination. I'll do so until > the end

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-07 Thread Helge Kreutzmann
Hello Simon, On Tue, Aug 07, 2018 at 08:20:23PM +0100, Simon McVittie wrote: > On Mon, 06 Aug 2018 at 17:31:35 +0200, Helge Kreutzmann wrote: > > this change (requiring a DISPLAY=:0) is really suprising. I'v used su > > for ages and a simple xhost + (yes, I know that this has security > >

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-07 Thread Simon McVittie
Control: retitle 905409 util-linux: "su -" no longer copies DISPLAY and XAUTHORITY to child, but this is not documented I'm retitling the bug since the new su implementation does break patterns involving xhost that used to work, but it is not the xhost step that is affected, making the title

Processed: Re: Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-07 Thread Debian Bug Tracking System
Processing control commands: > retitle 905409 util-linux: "su -" no longer copies DISPLAY and XAUTHORITY to > child, but this is not documented Bug #905409 [util-linux] upgrade of util-linux and login break the xhost command for other users Changed Bug title to 'util-linux: "su -" no longer

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-05 Thread Davide Prina
Hi Andreas, On 05/08/2018 05:40, Andreas Henriksson wrote: $ xhost +si:localuser:temp $ su - temp You are right that the old su would copy over DISPLAY and XAUTHORITY environment variables, even when you tell it to give you a new clean environment. probably this modification must be

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-04 Thread Andreas Henriksson
Hello Davide Prina, Thanks for your bug report. (Reply inline below.) On Sat, Aug 04, 2018 at 08:59:29AM +0200, Davide Prina wrote: > Package: util-linux > Version: 2.32-0.1 > Severity: grave > Justification: renders package unusable > > Dear Maintainer, > > upgrading > * util-linux:amd64

Bug#905409: upgrade of util-linux and login break the xhost command for other users

2018-08-04 Thread Davide Prina
Package: util-linux Version: 2.32-0.1 Severity: grave Justification: renders package unusable Dear Maintainer, upgrading * util-linux:amd64 (2.32-0.1, 2.32-0.3) * login:amd64 (1:4.5-1, 1:4.5-1.1) do not let work anymore something like this: $ xhost +si:localuser:temp $ su - temp $ firefox &