[systemd-devel] pam: Don't use loginuid [was: Re: Fix PAM module to not clobber XDG_RUNTIME_DIR with su]

[Dr. Werner Fink]

On Tue, Nov 26, 2013 at 10:41:36AM +0000, Colin Guthrie wrote:
> 'Twas brillig, and Martin Pitt at 26/11/13 06:19 did gyre and gimble:
> > Hey Lennart,
> > 
> > Lennart Poettering [2013-11-26  5:12 +0100]:
> >> I implemented this now, using a different approach than Martin's
> >> original patch (i.e. I don't think it is a good idea to involve stat()
> >> here, instead let's just let logind pass all information to
> >> pam_systemd).
> > 
> > Thanks!
> 
> Indeed, thanks for this!
> 
> If anyone backports this fix to v208 (i.e. pre sd-bus) please share it
> here. I'll likely do it just to have the "upstream-blessed" fix, but
> doubt I'll get around it it until later in the week.

I've backported it.  But during tests I've found that it does not help
if the environment variable XDG_RUNTIME_DIR already exists before doing
su.  It will not unset but exported.

Werner

-- 
  "Having a smoking section in a restaurant is like having
          a peeing section in a swimming pool." -- Edward Burr