OpenSSH 2.5.0p1 vs. SA_RESTART

Todd C. Miller Todd.Miller at courtesan.com
Sun Feb 18 15:14:00 EST 2001


In message <Pine.LNX.4.21.0102181501520.751-100000 at mothra.mindrot.org>
	so spake Damien Miller (djm):

> You are correct and my patch is completely wrong. We should have 
> what Stevens suggests:
> 
> #if defined(SA_RESTART)
> 		if (sig == SIGCHLD)
> 			sa.sa_flags |= SA_RESTART;
> #endif
> #if defined(SA_INTERRUPT)
> 		if (sig == SIGALRM)
> 			sa.sa_flags |= SA_INTERRUPT;
> #endif

Yup, that looks correct to me.  The BSD man pages claim that
SA_RESTART is a Berkeley extension, which may explain why it is
missing (or just not defined by default) on some OSes.

 - todd





More information about the openssh-unix-dev mailing list