Fixed bug #69197 (openssl_pkcs7_sign handles default value incorrectly)

diff --git a/NEWS b/NEWS
index d248f86483f6c94b55b3688de537e908b2c5aac8..dd1a80e382f8e321ed6eb78b5e1d8e229afdebfb 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -5,6 +5,8 @@
 - OpenSSL
   . Fixed bugs #68853, #65137 (Buffered crypto stream data breaks IO polling
     in stream_select() contexts) (Chris Wright)
+  . Fixed bug #69197 (openssl_pkcs7_sign handles default value incorrectly)
+    (Daniel Lowrey)
 
 - Core:
   . Fixed bug #68917 (parse_url fails on some partial urls). (Wei Dai)

diff --git a/ext/openssl/openssl.c b/ext/openssl/openssl.c
index e78d3b15aa8e04f2c0032b6b1a4d44c945b0cff2..1603009709e46f83f03f38dda17e8fe37bc2e865 100644 (file)
--- a/ext/openssl/openssl.c
+++ b/ext/openssl/openssl.c
@@ -4279,7 +4279,7 @@ PHP_FUNCTION(openssl_pkcs7_sign)
        char * outfilename;     int outfilename_len;
        char * extracertsfilename = NULL; int extracertsfilename_len;
 
-       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ppZZa!|lp",
+       if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ppZZa!|lp!",
                                &infilename, &infilename_len, &outfilename, &outfilename_len,
                                &zcert, &zprivkey, &zheaders, &flags, &extracertsfilename,
                                &extracertsfilename_len) == FAILURE) {