|
 |
e9ca13 |
From 3804c0d00fa6e629173fb1c8c61f8f88d5fe39b9 Mon Sep 17 00:00:00 2001
|
|
|
e9ca13 |
From: Stanislav Malyshev <stas@php.net>
|
|
|
e9ca13 |
Date: Mon, 23 Jun 2014 00:19:37 -0700
|
|
|
e9ca13 |
Subject: [PATCH] Fix bug #67498 - phpinfo() Type Confusion Information Leak
|
|
|
e9ca13 |
Vulnerability
|
|
|
e9ca13 |
|
|
|
e9ca13 |
---
|
|
|
e9ca13 |
ext/standard/info.c | 8 ++++----
|
|
|
e9ca13 |
ext/standard/tests/general_functions/bug67498.phpt | 15 +++++++++++++++
|
|
|
e9ca13 |
2 files changed, 19 insertions(+), 4 deletions(-)
|
|
|
e9ca13 |
create mode 100644 ext/standard/tests/general_functions/bug67498.phpt
|
|
|
e9ca13 |
|
|
|
e9ca13 |
diff --git a/ext/standard/info.c b/ext/standard/info.c
|
|
|
e9ca13 |
index 03ced35..0626a70 100644
|
|
|
e9ca13 |
--- a/ext/standard/info.c
|
|
|
e9ca13 |
+++ b/ext/standard/info.c
|
|
|
e9ca13 |
@@ -863,16 +863,16 @@ PHPAPI void php_print_info(int flag TSRMLS_DC)
|
|
|
e9ca13 |
|
|
|
e9ca13 |
php_info_print_table_start();
|
|
|
e9ca13 |
php_info_print_table_header(2, "Variable", "Value");
|
|
|
e9ca13 |
- if (zend_hash_find(&EG(symbol_table), "PHP_SELF", sizeof("PHP_SELF"), (void **) &data) != FAILURE) {
|
|
|
e9ca13 |
+ if (zend_hash_find(&EG(symbol_table), "PHP_SELF", sizeof("PHP_SELF"), (void **) &data) != FAILURE && Z_TYPE_PP(data) == IS_STRING) {
|
|
|
e9ca13 |
php_info_print_table_row(2, "PHP_SELF", Z_STRVAL_PP(data));
|
|
|
e9ca13 |
}
|
|
|
e9ca13 |
- if (zend_hash_find(&EG(symbol_table), "PHP_AUTH_TYPE", sizeof("PHP_AUTH_TYPE"), (void **) &data) != FAILURE) {
|
|
|
e9ca13 |
+ if (zend_hash_find(&EG(symbol_table), "PHP_AUTH_TYPE", sizeof("PHP_AUTH_TYPE"), (void **) &data) != FAILURE && Z_TYPE_PP(data) == IS_STRING) {
|
|
|
e9ca13 |
php_info_print_table_row(2, "PHP_AUTH_TYPE", Z_STRVAL_PP(data));
|
|
|
e9ca13 |
}
|
|
|
e9ca13 |
- if (zend_hash_find(&EG(symbol_table), "PHP_AUTH_USER", sizeof("PHP_AUTH_USER"), (void **) &data) != FAILURE) {
|
|
|
e9ca13 |
+ if (zend_hash_find(&EG(symbol_table), "PHP_AUTH_USER", sizeof("PHP_AUTH_USER"), (void **) &data) != FAILURE && Z_TYPE_PP(data) == IS_STRING) {
|
|
|
e9ca13 |
php_info_print_table_row(2, "PHP_AUTH_USER", Z_STRVAL_PP(data));
|
|
|
e9ca13 |
}
|
|
|
e9ca13 |
- if (zend_hash_find(&EG(symbol_table), "PHP_AUTH_PW", sizeof("PHP_AUTH_PW"), (void **) &data) != FAILURE) {
|
|
|
e9ca13 |
+ if (zend_hash_find(&EG(symbol_table), "PHP_AUTH_PW", sizeof("PHP_AUTH_PW"), (void **) &data) != FAILURE && Z_TYPE_PP(data) == IS_STRING) {
|
|
|
e9ca13 |
php_info_print_table_row(2, "PHP_AUTH_PW", Z_STRVAL_PP(data));
|
|
|
e9ca13 |
}
|
|
|
e9ca13 |
php_print_gpcse_array(ZEND_STRL("_REQUEST") TSRMLS_CC);
|
|
|
e9ca13 |
diff --git a/ext/standard/tests/general_functions/bug67498.phpt b/ext/standard/tests/general_functions/bug67498.phpt
|
|
|
e9ca13 |
new file mode 100644
|
|
|
e9ca13 |
index 0000000..5b5951b
|
|
|
e9ca13 |
--- /dev/null
|
|
|
e9ca13 |
+++ b/ext/standard/tests/general_functions/bug67498.phpt
|
|
|
e9ca13 |
@@ -0,0 +1,15 @@
|
|
|
e9ca13 |
+--TEST--
|
|
|
e9ca13 |
+phpinfo() Type Confusion Information Leak Vulnerability
|
|
|
e9ca13 |
+--FILE--
|
|
|
e9ca13 |
+
|
|
|
e9ca13 |
+$PHP_SELF = 1;
|
|
|
e9ca13 |
+phpinfo(INFO_VARIABLES);
|
|
|
e9ca13 |
+
|
|
|
e9ca13 |
+?>
|
|
|
e9ca13 |
+==DONE==
|
|
|
e9ca13 |
+--EXPECTF--
|
|
|
e9ca13 |
+phpinfo()
|
|
|
e9ca13 |
+
|
|
|
e9ca13 |
+PHP Variables
|
|
|
e9ca13 |
+%A
|
|
|
e9ca13 |
+==DONE==
|
|
|
e9ca13 |
--
|
|
|
e9ca13 |
1.9.2
|
|
|
e9ca13 |
|