6.0.0-beta6
4/9/26

[#9191] XSS Vulnerability
Summary XSS Vulnerability
Queue Gollem
Queue Version 1.1.1
Type Bug
State Resolved
Priority 3. High
Owners slusarz (at) horde (dot) org
Requester nightmare.lmw (at) anarchynet (dot) org
Created 8/21/10 (5710 days ago)
Due
Updated 10/12/22 (1275 days ago)
Assigned
Resolved 8/24/10 (5707 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
08/24/2010 06:38:26 PM Michael Slusarz Comment #3
Assigned to Michael Slusarz
State ⇒ Resolved
Reply to this comment
Git master fix:
http://lists.horde.org/archives/commits/2010-August/004747.html

This has been fixed in 1.1.2, although slightly different from your 
patch - we instead use the Horde::fatal() function which is the 
preferred way of reporting these kind of errors anyway.

Thank you for your report.
08/21/2010 02:20:21 PM nightmare (dot) lmw (at) anarchynet (dot) org Comment #1
Priority ⇒ 3. High
New Attachment: view.php.patched Download
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Gollem
Summary ⇒ XSS Vulnerability
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
I have found a Cross Site Scripting vulnerability in Gollem,

Exploit : 
http://localhost/horde/gollem/view.php?actionID=view_file&type=txt&file=<!--a75c305b1c0a6022--><script>alert("XSS")</script>&dir=../baddir/&driver=file

Vulnerable file : view.php (Line 32 - 46)

Vulnerable code :

if (is_callable(array($GLOBALS['gollem_vfs'], 'readStream'))) {
     $stream = $GLOBALS['gollem_vfs']->readStream($filedir, $filename);
     if (is_a($stream, 'PEAR_Error')) {
         Horde::logMessage($stream, __FILE__, __LINE__, PEAR_LOG_NOTICE);
         printf(_("Access denied to %s"), $filename);
         exit;
     }
} else {
     $data = $GLOBALS['gollem_vfs']->read($filedir, $filename);
     if (is_a($data, 'PEAR_Error')) {
         Horde::logMessage($data, __FILE__, __LINE__, PEAR_LOG_NOTICE);
         printf(_("Access denied to %s"), $filename);
         exit;
     }
}

I hope you fix the vulnerability asap. Patch in attachment.

Have a nice day.

Nicolas C. [NightMareLmW From DevSec]

Saved Queries