[Full-disclosure] Advisory 22/2005: Multiple vulnerabilities in phpSysInfo

2005-11-14T00:00:00
ID SECURITYVULNS:DOC:10214
Type securityvulns
Reporter Securityvulns
Modified 2005-11-14T00:00:00

Description

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1

                    Hardened PHP Project
                    www.hardened-php.net

                  -= Security  Advisory =-


 Advisory: Multiple vulnerabilities in phpSysInfo

Release Date: 2005/11/13 Last Modified: 2005/11/12 Author: Christopher Kunz <christopher.kunz@hardened-php.net> Application: phpSysInfo 2.4 and prior Severity: Cross-Site Scripting, HTTP Response Splitting, Arbitrary local file inclusion Risk: Medium Vendor Status: Vendor has released an updated version. References: http://www.hardened-php.net/advisory_212005.81.html

Overview:

phpSysInfo is a PHP script that displays information about the host being accessed. It will displays things like Uptime, CPU, Memory, SCSI, IDE, PCI, Ethernet, Floppy, and Video Information. A number of holes - all of them attributed to a rather crude (and unneeded) register_globals emulation - allow for arbitrary file inclusion, amongst other things.

Details:

It is important to note that due to the nature of phpSysInfo, it is unlikely to be run inside an open_basedir, since most of the information the software obtains is from the /proc or /etc directories. Furthermore, few admin- istrators will restrict its access to PHP's shell functions, since a such restriction would render the software unusable. In fact, the author requires that safe_mode be set to Off and no other restrictions be imposed on the PHP installation.

A hole that was reported in CVE-2003-0536 and is reported as "fixed" in the phpSysInfo README still persists. The reason for this problem lies in the incorrect handling of variables by the "globalization layer" implemented in phpSysInfo's index.php - quote: if (!empty($HTTP_GET_VARS)) while (list($name, $value) = each($HTTP_GET_VARS)) $$name = $value; if (!empty($HTTP_POST_VARS)) while (list($name, $value) = each($HTTP_POST_VARS)) $$name = $value; As the interested reader will easily see, this two-liner overwrites the complete scope, including the superglobal variable $_SERVER. By injecting a local path name into $_SERVER['HTTP_ACCEPT_LANGUAGE'], an attacker can now include arbitrary files. If they have a way to create content on the target server (such as an entry in syslog or the web server log file), they can use this to execute arbitrary code. In the Debian Sarge package, injecting only HTTP_ACCEPT_LANGUAGE is suffi- cient - the current source tree (phpsysinfo-dev from sf.net) requires the attacker to additionally inject the $lng parameter. Example: /index.php?_SERVER[HTTP_ACCEPT_LANGUAGE]=../../README%00 /index.php?_SERVER[HTTP_ACCEPT_LANGUAGE]=../../README%00&lng=../../ README%00

The variable $sensor_program is set in config.php, but can be overwritten since config.php is actually included before reglobalization takes place. With this variable, arbitrary file inclusion is possible on some platforms, dependant on the local realpath() implementation. Example: /index.php?sensor_program=lmsensors.inc.php/../../README%00

Additionally, $sensor_program can still be used to inject active contents into the page, known as Cross-Site Scripting. This issue was discussed in CVE-2005-0870 and still exists. The other XSS holes mentioned in that ad- visory are still valid, too. They can be abused with register_globals Off, this is different from the advisory by ISS. It is, however, not exploitable on Debian systems. An additional attack vector for Cross-Site Scripting attacks is the variable $VERSION, which is used for the version string displayed on the bottom of each page. Example: /index.php?VERSION=%22%3E%3Cscript%3Ealert('xss')%3C/script%3E

A fairly unknown attack class is so-called HTTP Response Splitting, an at- tack that allows for "selective defacement" of web pages by poisoning for- warding or reverse proxies. An explanation of this attack class is beyond the scope of this advisory, a whitepaper can be obtained at [1]. Using HTTP Response Splitting, arbitrary strings can be injected into the variable $charset, which is meant to include a value such as "iso-8859-1" or similar, but is only set to a value inside a language include file if a language in fact requires a character set different from iso-8859-1. In all other cases, it can be set via the URL. This variable is fed to a header() call without any additional checks. By breaking up the argument with \r\n, the attacker can inject a complete second HTTP response. This response is the only one that will be returned by any intermediate proxy, showing what- ever HTML the attacker injected previously. Example: /index.php?charset=%0d%0aContent-Length:%200%0d%0a%0d%0aHTTP/1.1%20200%20OK% 0d%0aContent-Type:%20text/html%0d%0aContent-Length:%2019%0d%0a%0d%0a<html>Ha cked!</html> This example will print out a simple (and invalid) HTML page containing only the string "Hacked!" if the victim accesses the phpSysInfo instance from be- hind a proxy. Direct access will probably yield an empty page, since many browsers do not know how to handle more than one response to a HTTP request.

Proof of Concept:

The examples above should provide you with enough PoC to validate the ex- istance of the vulnerabilities.

Disclosure Timeline:

  1. November 2005 - Bug disclosed to vendor.
  2. November 2005 - Updated version 2.4 released.
  3. November 2005 - 2.4 does not seem to fix - own patch sent to vendor.
  4. November 2005 - New updated version 2.4.1 fixes the issues.
  5. November 2005 - Public disclosure.

Credits:

All vulnerabilities were investigated by the Hardened-PHP Project. Credit to referred vulnerabilities goes to the original authors, as mentioned on CVE.

Recommendation:

We have provided a bugfix to the vendor, which, in addition to their own fix, will be included in version 2.4.1 of phpSysInfo. We also recommend installing the Hardening Patch for PHP which would have mitigated the response splitting problem (by not allowing multiline headers) and the $_SERVER overwriting issues (by not allowing certain superglobals to be overwritten from the outside). If you want to fix the issue yourself, just comment out the lines mentioned in an earlier paragraph of this advisory and selectively infer the variables $_REQUEST['lng'] and $_REQUEST['template'] from the outside. Additionally, access to the includes/ directory should be restricted by appropriate means, e.g. a .htaccess file.

CVE Information:

The Common Vulnerabilities and Exposures project (cve.mitre.org) has assigned the name CVE-2005-3347 to the file inclusion issue and the name name CVE-2005-3348 to the HTTP response splitting problem.

References:

[1] http://www.packetstormsecurity.org/papers/general/whitepaper_httprespons e.pdf [2] http://phpsysinfo.sourceforge.net/

Plug:

You can discuss this and other vulnerabilities in our forum at http://forum.hardened-php.net/ - an up-to-date list of advisories can be found at http://www.hardened-php.net/.

GPG-Key:

http://www.hardened-php.net/hardened-php-signature-key.asc

pub 1024D/0A864AA1 2004-04-17 Hardened-PHP Signature Key Key fingerprint = 066F A6D0 E57E 9936 9082 7E52 4439 14CC 0A86 4AA1

Copyright 2005 Christopher Kunz / Hardened PHP Project. All rights reserved.

-----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.6 (GNU/Linux) Comment: For info see http://www.gnupg.org

iD8DBQFDdyOFRDkUzAqGSqERAt9KAJ4uCXhrcapN/zxWDS6n7y8ezp6xsQCgw28i HSdHc4zKJ8aCj5gSC8Zxlj4= =XRw5 -----END PGP SIGNATURE----- ___________ Full-Disclosure - We believe in it. Charter: http://lists.grok.org.uk/full-disclosure-charter.html Hosted and sponsored by Secunia - http://secunia.com/