summaryrefslogtreecommitdiff
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
commit4ca32c31a0f842f2980c2d6a6584d6f680bfe839 (patch)
treed0b0ca18f588adb1790d9a4bb8cb0ecfd5f02429
parent26dc3f37c33bbeff57ffee46c0310848862e0900 (diff)
downloadnginx-4ca32c31a0f842f2980c2d6a6584d6f680bfe839.tar.gz
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;
}