Commit Diff

Commit:
a91ad7f2ffac3f1cec0c6c42e780ab5efc92ba5c
From:
Omar Polo <op@omarpolo.com>
Date:
Message:
drop unnecessary bzero the whole struct client is already memset'd to 0 in do_accept. handle_handshake doesn't touch the request or iri buffer in the code path that leads to handle_open_conn. (It does so in the error router alone.)
Actions:
Patch | Tree
commit - 79288c8b6077a573243c4654e3b3c4948febdb99
commit + a91ad7f2ffac3f1cec0c6c42e780ab5efc92ba5c
blob - 0ee226639fdaf684eac3b7c3236151b3eb59b4e9
blob + 14991b895c3755e72838aa9d890d774eca4f665b
--- server.c
+++ server.c
@@ -650,9 +650,6 @@ handle_open_conn(int fd, short ev, void *d)
 	const char *parse_err = "invalid request";
 	char decoded[DOMAIN_NAME_LEN];
 
-	bzero(c->req, sizeof(c->req));
-	bzero(&c->iri, sizeof(c->iri));
-
 	switch (tls_read(c->ctx, c->req, sizeof(c->req)-1)) {
 	case -1:
 		log_err(c, "tls_read: %s", tls_error(c->ctx));