# HG changeset patch # User Piotr Sikora # Date 1368743833 25200 # Node ID 83d028011ae2be9866a30153200afca94f472857 # Parent c248b00715071d4f8aec3e4e5d21cd1652c800a6 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 diff -r c248b0071507 -r 83d028011ae2 src/event/ngx_event_openssl_stapling.c --- a/src/event/ngx_event_openssl_stapling.c Thu Apr 25 17:41:45 2013 +0400 +++ b/src/event/ngx_event_openssl_stapling.c Thu May 16 15:37:13 2013 -0700 @@ -611,15 +611,14 @@ != 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; }