Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 86478 - dev-php/phpsysinfo: XSS and path disclosure issues
Summary: dev-php/phpsysinfo: XSS and path disclosure issues
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Security
Classification: Unclassified
Component: Vulnerabilities (show other bugs)
Hardware: All All
: High minor
Assignee: Gentoo Security
URL: http://www.securityfocus.com/archive/...
Whiteboard: B4 [noglsa] jaervosz
Keywords:
: 93099 (view as bug list)
Depends on:
Blocks:
 
Reported: 2005-03-24 00:15 UTC by Sune Kloppenborg Jeppesen (RETIRED)
Modified: 2005-05-20 01:52 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sune Kloppenborg Jeppesen (RETIRED) gentoo-dev 2005-03-24 00:15:34 UTC
XSS issue and path disclosure reported.
Comment 1 Thierry Carrez (RETIRED) gentoo-dev 2005-03-25 05:45:32 UTC
Ccing PHP/sebastian as maintainers for input

Upstream doesn't look very alive. We could reproduce, doublecheck the patch (see URL) and apply it... or get rid of that package.
Comment 2 Thierry Carrez (RETIRED) gentoo-dev 2005-03-30 07:18:52 UTC
PHP herd (or sebastian), please comment on comment #1
Comment 3 Sebastian Bergmann (RETIRED) gentoo-dev 2005-03-30 23:57:51 UTC
Sorry for not responding earlier to this.

I do not know what to do about this. I have never used phpsysinfo, do not know whether or not UPSTREAM is alive and cannot check the proposed fix for the problem.

Putting phpsysinfo into package.mask until such time that UPSTREAM resolves the issue sounds good to me.
Comment 4 Thierry Carrez (RETIRED) gentoo-dev 2005-03-31 00:18:17 UTC
sebastian: sorry, I misread the Changelog, thinking you were the recent bumper.
Cc-ing cryos and corsair as they look more like it.

Bug has been submitted a week ago upstream at:
http://sourceforge.net/tracker/index.php?func=detail&aid=1168383&group_id=15&atid=100015

PHP/cryos/corsair: if you agree with sebastian, please mask the package and comment here.
Comment 5 Markus Rothe (RETIRED) gentoo-dev 2005-04-01 09:14:46 UTC
<<< phpsysinfo-2.3-r2.ebuild
<<< files/phpsysinfo-2.3-xss-and-path.patch
<<< files/digest-phpsysinfo-2.3-r2

Koon: I've marked phpsysinfo-2.3-r2 stable on ppc64. other arches should follow.
Comment 6 Thierry Carrez (RETIRED) gentoo-dev 2005-04-01 13:30:09 UTC
TARGET KEYWORDS="x86 ppc ppc64 alpha hppa sparc amd64"
Arches, please test and mark stable
Comment 7 Clemens Noss 2005-04-01 14:20:00 UTC
The patch doesn't apply. It contains this line (for system_footer.php):

 // $Id: phpsysinfo-2.3-xss-and-path.patch,v 1.1 2005/04/01 17:09:08 corsair Exp $

but phpsysinfo-dev/includes/system_footer.php contains

// $Id: system_footer.php,v 1.38 2004/08/13 23:02:32 webbie Exp $
Comment 8 Markus Rothe (RETIRED) gentoo-dev 2005-04-01 23:36:48 UTC
ok.. I was some kind of clueless, but I found the problem with the patch:

My local copy works like a charm, but as soon as I commit the patch it get corruted due to cvs.

The patch look like this:

[...]
--- includes/os/class.Darwin.inc.php.orig   2005-04-02 06:48:39.000000000 +0000
+++ includes/os/class.Darwin.inc.php    2005-03-23 16:40:36.000000000 +0000
@@ -19,8 +19,13 @@

 // $Id: class.Darwin.inc.php,v 1.16 2004/06/26 23:46:36 webbie Exp $

+if(!file_exists('./includes/os/class.BSD.common.inc.php')) {
+    exit;
+}
+
 require('./includes/os/class.BSD.common.inc.php');

+
 echo "<p align=center><b>Note: The Darwin version of phpSysInfo is work in progress, some things currently don't work</b></p>";

 class sysinfo extends bsd_common {
[...]

Now there is that line: 

$Id: class.Darwin.inc.php,v 1.16 2004/06/26 23:46:36 webbie Exp $

And cvs commit changes the patch to this:

[...]
--- includes/os/class.Darwin.inc.php.orig   2005-04-02 06:48:39.000000000 +0000
+++ includes/os/class.Darwin.inc.php    2005-03-23 16:40:36.000000000 +0000
@@ -19,8 +19,13 @@

 // $Id: phpsysinfo-2.3-xss-and-path.patch,v 1.3 2005/04/02 07:28:50 corsair Exp $

+if(!file_exists('./includes/os/class.BSD.common.inc.php')) {
+    exit;
+}
+
 require('./includes/os/class.BSD.common.inc.php');

+
 echo "<p align=center><b>Note: The Darwin version of phpSysInfo is work in progress, some things currently don't work</b></p>";

 class sysinfo extends bsd_common {
[...]

The result is that the patch cannot be applied. I'm currently checking how to avoid this..
Comment 9 Markus Rothe (RETIRED) gentoo-dev 2005-04-02 00:02:40 UTC
I
Comment 10 Markus Rothe (RETIRED) gentoo-dev 2005-04-02 00:02:40 UTC
I´ve mirrored the patch on dev.gentoo.org. ebuild should now be correct.
Comment 11 Jason Wever (RETIRED) gentoo-dev 2005-04-02 09:41:33 UTC
Stable on SPARC.
Comment 12 Michael Hanselmann (hansmi) (RETIRED) gentoo-dev 2005-04-02 13:13:13 UTC
Stable on ppc.
Comment 13 Bryan Østergaard (RETIRED) gentoo-dev 2005-04-03 03:01:22 UTC
Alpha stable.
Comment 14 Marcus D. Hanwell (RETIRED) gentoo-dev 2005-04-04 05:49:03 UTC
Sorry - been really busy. I have been testing the new version, but am suffering from this bug - http://sourceforge.net/tracker/index.php?func=detail&aid=1094032&group_id=15&atid=100015 as reported upstream. I get a blank page with only <html><body></body></html>. Not sure if this is new as I haven't used it in quite a while. Anyone else on amd64 running this version successfully? This is with PHP 5.0.3-r2.
Comment 15 Thierry Carrez (RETIRED) gentoo-dev 2005-04-06 00:51:26 UTC
I vote NO GLSA btw: phpsysinfo shouldn't be available to unknown people, and path disclosure is the least of your worries in this case.
Comment 16 Guy Martin (RETIRED) gentoo-dev 2005-04-08 11:50:39 UTC
Stable on hppa.
Comment 17 Luke Macken (RETIRED) gentoo-dev 2005-04-10 19:27:36 UTC
I vote no GLSA as well.
Comment 18 Sune Kloppenborg Jeppesen (RETIRED) gentoo-dev 2005-04-11 23:04:12 UTC
Ok, then we only need stable marking on x86 and amd64 before we can close this bug.
Comment 19 Olivier Crete (RETIRED) gentoo-dev 2005-04-11 23:26:03 UTC
x86 there
Comment 20 Simon Stelling (RETIRED) gentoo-dev 2005-04-12 09:05:26 UTC
amd64 done
Comment 21 Sune Kloppenborg Jeppesen (RETIRED) gentoo-dev 2005-04-12 09:39:27 UTC
Thx everyone. Closing without GLSA.
Comment 22 Thierry Carrez (RETIRED) gentoo-dev 2005-05-20 01:52:26 UTC
*** Bug 93099 has been marked as a duplicate of this bug. ***