From dd5479ea4c0f6b3dcb57af2b877e6b4bb2a0b319 Mon Sep 17 00:00:00 2001 From: Anatol Belski Date: Fri, 6 May 2016 09:19:04 +0200 Subject: Revert "Fixed bug #72165 Null pointer dereference - openssl_csr_new" This reverts commit 7277c85765d1053c8cd1d1093902df541b3d101a. --- ext/openssl/openssl.c | 12 +++--------- ext/openssl/tests/bug72165.phpt | 17 ----------------- 2 files changed, 3 insertions(+), 26 deletions(-) delete mode 100644 ext/openssl/tests/bug72165.phpt diff --git a/ext/openssl/openssl.c b/ext/openssl/openssl.c index 44b3fa4985..4a096f779a 100644 --- a/ext/openssl/openssl.c +++ b/ext/openssl/openssl.c @@ -2764,18 +2764,12 @@ static int php_openssl_make_REQ(struct php_x509_request * req, X509_REQ * csr, z } } if (attribs) { - zend_long numindex; - ZEND_HASH_FOREACH_STR_KEY_VAL(Z_ARRVAL_P(attribs), numindex, strindex, item) { + ZEND_HASH_FOREACH_STR_KEY_VAL(Z_ARRVAL_P(attribs), strindex, item) { int nid; convert_to_string_ex(item); - if (NULL == strindex) { - char tmp[ZEND_LTOA_BUF_LEN]; - ZEND_LTOA(numindex, tmp, ZEND_LTOA_BUF_LEN); - nid = OBJ_txt2nid(tmp); - } else { - nid = OBJ_txt2nid(ZSTR_VAL(strindex)); - } + + nid = OBJ_txt2nid(ZSTR_VAL(strindex)); if (nid != NID_undef) { if (!X509_NAME_add_entry_by_NID(subj, nid, MBSTRING_UTF8, (unsigned char*)Z_STRVAL_P(item), -1, -1, 0)) { php_error_docref(NULL, E_WARNING, "attribs: add_entry_by_NID %d -> %s (failed)", nid, Z_STRVAL_P(item)); diff --git a/ext/openssl/tests/bug72165.phpt b/ext/openssl/tests/bug72165.phpt deleted file mode 100644 index c7e0d1a7c6..0000000000 --- a/ext/openssl/tests/bug72165.phpt +++ /dev/null @@ -1,17 +0,0 @@ ---TEST-- -Bug #72165 Null pointer dereference - openssl_csr_new ---SKIPIF-- - ---FILE-- - "hello", 1 => "world"); -$var2 = openssl_csr_new(array(0),$var0,null,array(0)); -?> -==DONE== ---EXPECTF-- -Warning: openssl_csr_new(): dn: is not a recognized name in %sbug72165.php on line %d - -Warning: openssl_csr_new(): add1_attr_by_txt challengePassword_min -> 4 (failed; check error queue and value of string_mask OpenSSL option if illegal characters are reported) in %sbug72165.php on line %d -==DONE== -- cgit v1.2.1