summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorPiotr Sikora <piotr@cloudflare.com>2013-05-16 15:37:13 -0700
committerPiotr Sikora <piotr@cloudflare.com>2013-05-16 15:37:13 -0700
commit1b10a9adf8447145ac2607f16681340049409fc8 (patch)
treebe492fadb98fe7fd5c2fb0511a6736c668848975
parent8597218f386351d6c6cdced24af6716e19a18fc3 (diff)
downloadnginx-1b10a9adf8447145ac2607f16681340049409fc8.tar.gz
nginx-1b10a9adf8447145ac2607f16681340049409fc8.tar.bz2
OCSP stapling: fix error logging of successful OCSP responses.
Due to a bad argument list, nginx worker would crash (SIGSEGV) while trying to log the fact that it received OCSP response with "revoked" or "unknown" certificate status. While there, fix similar (but non-crashing) error a few lines above. Signed-off-by: Piotr Sikora <piotr@cloudflare.com>
-rw-r--r--src/event/ngx_event_openssl_stapling.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/src/event/ngx_event_openssl_stapling.c b/src/event/ngx_event_openssl_stapling.c
index aaa8d8ac4..77baeb98f 100644
--- a/src/event/ngx_event_openssl_stapling.c
+++ b/src/event/ngx_event_openssl_stapling.c
@@ -611,15 +611,14 @@ ngx_ssl_stapling_ocsp_handler(ngx_ssl_ocsp_ctx_t *ctx)
!= 1)
{
ngx_log_error(NGX_LOG_ERR, ctx->log, 0,
- "certificate status not found in the OCSP response",
- n, OCSP_response_status_str(n));
+ "certificate status not found in the OCSP response");
goto error;
}
if (n != V_OCSP_CERTSTATUS_GOOD) {
ngx_log_error(NGX_LOG_ERR, ctx->log, 0,
"certificate status \"%s\" in the OCSP response",
- n, OCSP_cert_status_str(n));
+ OCSP_cert_status_str(n));
goto error;
}