commit a91ad7f2ffac3f1cec0c6c42e780ab5efc92ba5c from: Omar Polo date: Fri Sep 24 08:08:49 2021 UTC 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.) 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));