ControlMaster cwd is not /
ssh at raf.org
Fri Apr 9 14:36:27 AEST 2021
On Fri, Apr 09, 2021 at 01:41:47PM +1000, raf <ssh at raf.org> wrote:
> On Thu, Apr 08, 2021 at 02:48:59PM +0200, Thorsten Glaser <t.glaser at tarent.de> wrote:
> > On Thu, 8 Apr 2021, Bert Wesarg wrote:
> > > see
> > > https://bugzilla.mindrot.org/show_bug.cgi?id=1902
> > > for one of the first reports and also a reason why it is currently not accepted.
> > This seems really wrong. #1902 refers to #1988 as reason for not
> > chdir’ing to / but #1988 is about (1, 0) vs. (1, 1) whereas #1902
> > is about (0, 1) vs. (1, 1) so perhaps changing the daemon call to
> > (0, 0) will fix both?
> > bye,
> > //mirabilos, not (yet) having looked at that code, just wondering
> Ah, #1988 is the reason. I would think that if that problem
> was that stderr is left open, the solution is to close stderr
> (when not debugging), rather than to not chdir /.
The current code has daemon(1, 1) in control_persist_detach().
The stderr problem is handled separately. So I think it needs
to be changed to daemon(0, 1). I've created a new pull request
to do it that way:
More information about the openssh-unix-dev