diff options
| author |   ray <ray@openbsd.org> | 2007-05-26 23:52:04 +0000 |
|---|---|---|
| committer | ray <ray@openbsd.org> | 2007-05-26 23:52:04 +0000 |
| commit | ffd4dac3becd034263ccfae9bdb3ee9f79ad3e52 (patch) | |
| tree | 257595b64c71f4b7e4cd0d57f90a68fcb36e10f6 | |
| parent | It is too complicated to tell a client that an internal failure (diff) | |
| download | wireguard-openbsd-ffd4dac3becd034263ccfae9bdb3ee9f79ad3e52.tar.xz wireguard-openbsd-ffd4dac3becd034263ccfae9bdb3ee9f79ad3e52.zip | |
It is not the job of cvs_server_send_response() to do error checking.
Also remove unused variable.
OK niallo@
| -rw-r--r-- | usr.bin/cvs/server.c | 18 |
1 files changed, 2 insertions, 16 deletions
diff --git a/usr.bin/cvs/server.c b/usr.bin/cvs/server.c index 22a986d905a..1555b5f9fd3 100644 --- a/usr.bin/cvs/server.c +++ b/usr.bin/cvs/server.c @@ -1,4 +1,4 @@ -/* $OpenBSD: server.c,v 1.58 2007/05/26 23:19:31 ray Exp $ */ +/* $OpenBSD: server.c,v 1.59 2007/05/26 23:52:04 ray Exp $ */ /* * Copyright (c) 2006 Joris Vink <joris@openbsd.org> * @@ -126,27 +126,13 @@ void cvs_server_send_response(char *fmt, ...) { va_list ap; - char *data, *s; - struct cvs_resp *resp; + char *data; va_start(ap, fmt); if (vasprintf(&data, fmt, ap) == -1) fatal("vasprintf: %s", strerror(errno)); va_end(ap); - if ((s = strchr(data, ' ')) != NULL) - *s = '\0'; - - resp = cvs_remote_get_response_info(data); - if (resp == NULL) - fatal("'%s' is an unknown response", data); - - if (resp->supported != 1) - fatal("remote cvs client does not support '%s'", data); - - if (s != NULL) - *s = ' '; - cvs_log(LP_TRACE, "%s", data); cvs_remote_output(data); xfree(data); |