[PATCH] Fix minor breakage on Cygwin: auth-passwd.c andsession.c

Markus Friedl markus at openbsd.org
Tue Aug 12 23:42:08 EST 2003


On Tue, Aug 12, 2003 at 03:17:57PM +0200, Corinna Vinschen wrote:
>         /* we will sleep from 500ms to 2500ms */
>         break_length = MIN(break_length, 2500);
>         break_length = MAX(break_length,  500);
>         tcsendbreak(s->ttyfd, 0);
>         /* should we care about EINTR? */
>         usleep(break_length * 1000);
>         return 1;

i really don't want to sleep.

there reason we sleep in the original code is
that the draft specifies the duration of the
break, if we just use tcsendbreak() then we
ignore the specified duration, so no additional sleep
is necessary.

on the otherhand, i don't see a need for the break length,
this should be tcsendbreak()'s job.




More information about the openssh-unix-dev mailing list