lightdm should not rename() .xsession-errors, as it may be a symlink
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
lightdm (Ubuntu) |
Confirmed
|
Low
|
Unassigned |
Bug Description
http://
Lightdm calls rename() on the xsession-errors file. In our deployment many of our users have this file as a symlink. Their homedirectory is on NFS and it will quickly fill their quota with bogus messages.
You should lstat() the file first. If it is a symlink, you can make a new file at the destination and update the link. I can supply a patch if you wish.
antarus@
lightdm:
Installed: 1.2.1-0ubuntu1.1
Candidate: 1.2.1-0ubuntu1.1
Version table:
DISTRIB_
DISTRIB_
DISTRIB_ID=Ubuntu
DISTRIB_
-A
Thank you for your bug report, the people working on lightdm are pretty busy with other projects as well, having a patch up for review would be very welcome if you want to work on one ;-)
Note that the best to submit one for review is through a merge request, you can do one following those steps:
bzr branch lp:lightdm /lightdm/ xsession- errors- symlink- fix
cd lightdm
<do your changes>
bzr commit
bzr push lp:~<yourlpid>
bzr lp-open
then select "submit for review" on the page and file the description
Thanks in advance!