The branch master has been updated via 5fe499cb75469fbda08d96facd13d14a402a6d44 (commit) from 12df11bdf11fb6a3410483b0097f032e329b4623 (commit)
- Log ----------------------------------------------------------------- commit 5fe499cb75469fbda08d96facd13d14a402a6d44 Author: Viktor Dukhovni <openssl-us...@dukhovni.org> Date: Mon Jul 15 13:12:04 2019 -0400 Actually silently ignore GET / OCSP requests Reviewed-by: Matt Caswell <m...@openssl.org> ----------------------------------------------------------------------- Summary of changes: apps/ocsp.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/apps/ocsp.c b/apps/ocsp.c index 9f2cf45..71c6a56 100644 --- a/apps/ocsp.c +++ b/apps/ocsp.c @@ -1436,9 +1436,11 @@ static int do_responder(OCSP_REQUEST **preq, BIO **pcbio, BIO *acbio, *q = '\0'; /* - * Skip "GET / HTTP..." requests often used by load-balancers + * Skip "GET / HTTP..." requests often used by load-balancers. Note: + * 'p' was incremented above to point to the first byte *after* the + * leading slash, so with 'GET / ' it is now an empty string. */ - if (p[1] == '\0') + if (p[0] == '\0') goto out; len = urldecode(p);