Ludovic Courtès <l...@gnu.org> writes:
> Digging further, I fetched the source from > <https://packages.ubuntu.com/bionic/sudo>, and boom! I found the > culprit: it’s called ‘debian/patches/keep_home_by_default.patch’. > > --8<---------------cut here---------------start------------->8--- > Description: Set HOME in initial_keepenv_table > Set HOME in initial_keepenv_table; without this, $HOME will never be > preserved unless added to keep_env. There's appropriate logic to handle > resetting the home for -H and -i options, so this is the only part that's > missing. > Author: Steve Langasek <steve.langa...@canonical.com> > --- a/plugins/sudoers/env.c > +++ b/plugins/sudoers/env.c > @@ -189,6 +189,7 @@ > "COLORS", > "DISPLAY", > "DPKG_COLORS", > + "HOME", > "HOSTNAME", > "KRB5CCNAME", > "LS_COLORS", > --8<---------------cut here---------------end--------------->8--- > > (This patch is playing with fire IMO. If you’re an Ubuntu user, > consider reporting a bug!) Wow. Changing fundamental behaviour like that is bad. > We could apply the patch I posted earlier, which simply disables profile > migration when SUDO_USER is set. That won’t address the fact that root > writes to the user’s ~/.cache, but there’s not much we can do here. This sounds fine to me. -- Ricardo
