diff options
| author | Theo de Raadt <deraadt@cvs.openbsd.org> | 2004-12-04 18:00:44 +0000 |
|---|---|---|
| committer | Theo de Raadt <deraadt@cvs.openbsd.org> | 2004-12-04 18:00:44 +0000 |
| commit | 20eeebec9f2d042124148e79ecc26c802a32c317 (patch) | |
| tree | fed6362501fc362cc95dda390d5733c7ef466b99 /libexec/ftpd/ftpcmd.y | |
| parent | 574ecaad13b64447ec4608bb7d86cae17f7e9957 (diff) | |
reply() used to play a lot with stdout, expecially fflush(stdout).
The recent change is to avoid stdio. That's good, except there are
nearly 130 calls to this function, yet what if one of them depended on
fflush() or something else in this code? The semantic change was
never checked. That is not how we do development -- back this out
until we know that checking work has been done.
Diffstat (limited to 'libexec/ftpd/ftpcmd.y')
| -rw-r--r-- | libexec/ftpd/ftpcmd.y | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/libexec/ftpd/ftpcmd.y b/libexec/ftpd/ftpcmd.y index 1f2ab22fdb5..2f774c30882 100644 --- a/libexec/ftpd/ftpcmd.y +++ b/libexec/ftpd/ftpcmd.y @@ -1,4 +1,4 @@ -/* $OpenBSD: ftpcmd.y,v 1.45 2004/12/03 23:57:40 moritz Exp $ */ +/* $OpenBSD: ftpcmd.y,v 1.46 2004/12/04 18:00:43 deraadt Exp $ */ /* $NetBSD: ftpcmd.y,v 1.7 1996/04/08 19:03:11 jtc Exp $ */ /* @@ -44,7 +44,7 @@ static const char sccsid[] = "@(#)ftpcmd.y 8.3 (Berkeley) 4/6/94"; #else static const char rcsid[] = - "$OpenBSD: ftpcmd.y,v 1.45 2004/12/03 23:57:40 moritz Exp $"; + "$OpenBSD: ftpcmd.y,v 1.46 2004/12/04 18:00:43 deraadt Exp $"; #endif #endif /* not lint */ @@ -1170,6 +1170,7 @@ toolong(signo) { struct syslog_data sdata = SYSLOG_DATA_INIT; + /* XXX signal races */ reply(421, "Timeout (%d seconds): closing control connection.", timeout); if (logging) |