[PATCH]: Drop Cygwin workaround

Tim Rice tim at multitalents.net
Sat Sep 8 03:39:06 EST 2001


On Fri, 7 Sep 2001, Corinna Vinschen wrote:

> On Fri, Sep 07, 2001 at 08:12:16AM -0700, Tim Rice wrote:
> > On Thu, 30 Aug 2001, Corinna Vinschen wrote:
> >
> > > Hi,
> > >
> > > the Cygwin specific workaround to sleep a second in daemon() isn't
> > > needed anymore. We can eliminate it from the OpenSSH sources.
> >
> > Is it not needed because of changes to the OpenSSH code or
> > changes to Cygwin?
>
> Changes to Cygwin.

Then removing it would break on older versions of Cygwin.
Unless you have a configure time test for Cygwin ver >= n.nn
it doesn't seem worthwhile to remove the sleep(1).

>
> Corinna
>
> >
> > >
> > > Index: daemon.c
> > > ===================================================================
> > > RCS file: /cvs/openssh_cvs/openbsd-compat/daemon.c,v
> > > retrieving revision 1.1
> > > diff -u -p -r1.1 daemon.c
> > > --- daemon.c	2001/01/31 21:52:03	1.1
> > > +++ daemon.c	2001/08/30 08:41:39
> > > @@ -51,13 +51,6 @@ daemon(nochdir, noclose)
> > >  	case 0:
> > >  		break;
> > >  	default:
> > > -#ifdef HAVE_CYGWIN
> > > -		/*
> > > -		 * This sleep avoids a race condition which kills the
> > > -		 * child process if parent is started by a NT/W2K service.
> > > -		 */
> > > -		sleep(1);
> > > -#endif
> > >  		_exit(0);
> > >  	}
> > >
> > >
> > > Corinna
> > >
> > >
> >
> > --
> > Tim Rice				Multitalents	(707) 887-1469
> > tim at multitalents.net
> >
>
>

-- 
Tim Rice				Multitalents	(707) 887-1469
tim at multitalents.net





More information about the openssh-unix-dev mailing list