summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPierre Joye <pajoye@php.net>2006-11-15 23:28:06 +0000
committerPierre Joye <pajoye@php.net>2006-11-15 23:28:06 +0000
commitc412409164b0ca46fe1abfb1c2b16333d9b6ae8e (patch)
tree5432beb00a16cec30439e7f6c24633fe9ce37072
parentd1acb8d4ebd5b7bf2b7ec2e1f2a4b79a6f038f02 (diff)
downloadphp-git-c412409164b0ca46fe1abfb1c2b16333d9b6ae8e.tar.gz
- MFH: #39217, serialNumber might be -1 when the value is too big
-rw-r--r--NEWS2
-rw-r--r--ext/openssl/openssl.c3
2 files changed, 4 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index e161130ce2..0fedebc7e6 100644
--- a/NEWS
+++ b/NEWS
@@ -65,6 +65,8 @@ PHP NEWS
of overloaded array). (Dmitry)
- Fixed bug #39265 (Fixed path handling inside mod_files.sh).
(michal dot taborsky at gmail dot com, Ilia)
+- Fixed bug #39217 (serialNumber might be -1 when the value is too large)
+ (Pierre, Tony)
- Fixed bug #39215 (Inappropriate close of stdin/stdout/stderr). (Wez,Ilia)
- Fixed bug #39151 (Parse error in recursiveiteratoriterator.php). (Marcus)
- Fixed bug #39121 (Incorrect return array handling in non-wsdl soap client).
diff --git a/ext/openssl/openssl.c b/ext/openssl/openssl.c
index 889f40cedf..8515ba80b7 100644
--- a/ext/openssl/openssl.c
+++ b/ext/openssl/openssl.c
@@ -1003,7 +1003,8 @@ PHP_FUNCTION(openssl_x509_parse)
add_assoc_name_entry(return_value, "issuer", X509_get_issuer_name(cert), useshortnames TSRMLS_CC);
add_assoc_long(return_value, "version", X509_get_version(cert));
- add_assoc_long(return_value, "serialNumber", ASN1_INTEGER_get(X509_get_serialNumber(cert)));
+
+ add_assoc_string(return_value, "serialNumber", i2s_ASN1_INTEGER(NULL, X509_get_serialNumber(cert)), 1);
add_assoc_asn1_string(return_value, "validFrom", X509_get_notBefore(cert));
add_assoc_asn1_string(return_value, "validTo", X509_get_notAfter(cert));