[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
This bug was fixed in the package shadow - 1:4.2-3.1ubuntu5.4 --- shadow (1:4.2-3.1ubuntu5.4) xenial; urgency=medium * patches/1012_extrausers_chfn.patch: - add --extrausers option to "chfn" (LP: #1495580) * debian/patches/1013_extrausers_deluser.patch: - add --extrausers option to "userdel" (LP: #1659534) * debian/patches/2000_fix-su-pam-env-handling: - fix "su -l" to correctly use pam_getenvlist (LP: #984390) -- Michael Vogt Fri, 22 Mar 2019 20:22:06 +0100 ** Changed in: shadow (Ubuntu Xenial) Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: Fix Released Status in shadow source package in Bionic: Fix Released Bug description: TEST CASE: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
This bug was fixed in the package shadow - 1:4.5-1ubuntu2 --- shadow (1:4.5-1ubuntu2) bionic; urgency=medium * debian/patches/1013_extrausers_deluser.patch - add --extrausers option to "userdel" (LP: #1659534) * debian/patches/2000_fix-su-pam-env-handling. - fix "su -l" to correctly use pam_getenvlist (LP: #984390) -- Michael Vogt Fri, 22 Mar 2019 20:05:38 +0100 ** Changed in: shadow (Ubuntu Bionic) Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: Fix Released Status in shadow source package in Bionic: Fix Released Bug description: TEST CASE: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
SRU validated, no issues found. Logs: xenial: https://paste.ubuntu.com/p/9jdjdQns7t/ bionic: https://paste.ubuntu.com/p/P6CZ5WnmmB/ ** Tags removed: verification-needed verification-needed-bionic verification-needed-xenial ** Tags added: verification-done verification-done-bionic verification-done-xenial -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: Fix Committed Status in shadow source package in Bionic: Fix Committed Bug description: TEST CASE: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
** Description changed: + TEST CASE: + $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: Fix Committed Status in shadow source package in Bionic: Fix Committed Bug description: TEST CASE: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
** Changed in: shadow (Ubuntu Xenial) Importance: Undecided => Medium ** Changed in: shadow (Ubuntu Bionic) Importance: Undecided => Medium -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: Fix Committed Status in shadow source package in Bionic: Fix Committed Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
Hello James, or anyone else affected, Accepted shadow into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/shadow/1:4.2-3.1ubuntu5.4 in a few hours, and then in the -proposed repository. Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users. If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. In either case, without details of your testing we will not be able to proceed. Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping! N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days. ** Changed in: shadow (Ubuntu Xenial) Status: New => Fix Committed ** Tags added: verification-needed-xenial -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: Fix Committed Status in shadow source package in Bionic: Fix Committed Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
Hello James, or anyone else affected, Accepted shadow into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/shadow/1:4.5-1ubuntu2 in a few hours, and then in the -proposed repository. Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users. If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed. Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping! N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days. ** Changed in: shadow (Ubuntu Bionic) Status: New => Fix Committed ** Tags added: verification-needed verification-needed-bionic -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: New Status in shadow source package in Bionic: Fix Committed Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
This is "fixed" in disco - the "su" binary does no longer comes from "shadow" here but from util-linux. And there this bug does not exist. ** Changed in: shadow (Ubuntu) Status: Triaged => Fix Released -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Fix Released Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: New Status in shadow source package in Bionic: New Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
** Description changed: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games - | buildd@panlong:~$ cat /etc/environment + | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" - | buildd@panlong:~$ grep PATH /etc/login.defs + | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH - | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs + | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ + + REGRESSION POTENTIAL: + - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: New Status in shadow source package in Bionic: New Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
** Also affects: shadow (Ubuntu Xenial) Importance: Undecided Status: New ** Also affects: shadow (Ubuntu Bionic) Importance: Undecided Status: New ** Changed in: shadow (Ubuntu Precise) Status: Triaged => Won't Fix -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Won't Fix Status in shadow source package in Xenial: New Status in shadow source package in Bionic: New Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ REGRESSION POTENTIAL: - medium: it changes (incorrect) existing behaviour so https://xkcd.com/1172/ may apply To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
@mvo I'm ok with the cherry-pick since I'd like to move over the complete login binary package to util-linux in Debian before merging shadox + util-linux to disco. If you have already tested the cherry- picked version please go ahead otherwise i put it on my TODO list for the next weeks. -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Triaged Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
This is now fixed upstream: https://github.com/shadow- maint/shadow/pull/119 - so the next shadow release should have it. This makes me also wonder if we should simply cherry pick it for disco. -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Triaged Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
Fwiw, I looked at the code of "su.c" from util-linux (because debian/ubuntu may switch to it, see https://bugs.debian.org/cgi- bin/bugreport.cgi?bug=833256) and it looks like the bug is not present there. The su-common.c:modify_environment() code will first setenv_path() from login.defs and then do the supam_export_environment() which will override the existing PATH if /etc/environment also defines it. -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Triaged Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
I pushed a slightly modified version of this to https://github.com /shadow-maint/shadow/pull/119 ** Bug watch added: Debian Bug tracker #833256 https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=833256 -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Triaged Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
** Tags added: patch -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Triaged Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
[Touch-packages] [Bug 984390] Re: $PATH is taken from login.defs not /etc/environment
** Attachment added: "quilt style patch that fixes the issue" https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+attachment/4811188/+files/2000_fix-su-pam-env-handling -- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to shadow in Ubuntu. https://bugs.launchpad.net/bugs/984390 Title: $PATH is taken from login.defs not /etc/environment Status in shadow package in Ubuntu: Triaged Status in shadow source package in Precise: Triaged Bug description: $PATH isn't sourced from /etc/environment, instead the version in /etc/login.defs is used. (The example below comes from a precise install.) | james@panlong:~$ echo $PATH | /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ cat /etc/environment | PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games" | buildd@panlong:~$ grep PATH /etc/login.defs | # Three items must be defined: MAIL_DIR, ENV_SUPATH, and ENV_PATH. | # *REQUIRED* The default PATH settings, for superuser and normal users. | ENV_SUPATH PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin | ENV_PATH PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | #CRACKLIB_DICTPATH | buildd@panlong:~$ sudo sed -i -e "s#^ENV_PATH.*#ENV_PATH PATH=/wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games#" /etc/login.defs | buildd@panlong:~$ logout | james@panlong:~$ sudo su - buildd | buildd@panlong:~$ echo $PATH | /wtf:/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games | buildd@panlong:~$ To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/shadow/+bug/984390/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
