Bug 97585 - mail-client/squirrelmail vulnerability in options_identites.php (vendor-sec) (CAN-2005-2095)
Bug#: 97585 Product:  Gentoo Security Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: minor Priority: P2
Resolution: FIXED Assigned To: security@gentoo.org Reported By: jaervosz@gentoo.org
Component: Vulnerabilities
URL: 
Summary: mail-client/squirrelmail vulnerability in options_identites.php (vendor-sec) (CAN-2005-2095)
Keywords:  
Status Whiteboard: B4 [noglsa] jaervosz
Opened: 2005-07-01 01:39 0000
Description:   Opened: 2005-07-01 01:39 0000
Hello all,

A new vulnerability has been discovered in SquirrelMail. The file
src/options_identities.php contained some very bad, legacy code: an
extract($_POST) was done, effectively allowing a malicious attacker to
change session variables and even other people's preferences.

It must be noted that for this to happen you need to trick someone into
using an external form to post the information which is not trivial.

Affected versions:
1.4.0 - 1.4.5-RC1 

------- Comment #1 From Sune Kloppenborg Jeppesen 2005-07-01 01:39:53 0000 -------
Hello all,

A new vulnerability has been discovered in SquirrelMail. The file
src/options_identities.php contained some very bad, legacy code: an
extract($_POST) was done, effectively allowing a malicious attacker to
change session variables and even other people's preferences.

It must be noted that for this to happen you need to trick someone into
using an external form to post the information which is not trivial.

Affected versions:
1.4.0 - 1.4.5-RC1  (current stable tree)
1.2.8 - 1.2.10     (unsupported old stable tree)
1.5.x CVS          (unsupported current development tree)

Not vulnerable:
Everything before 1.2.8.

Our proposed patch is attached; unfortunately we had to rework some
functions to fix them the right way because the previous code really
depended to the extract() call.

We will release 1.4.5 sometime next week with the patch included. Fixes
for unsupported trees will be applied to their CVS branches but no new
releases will be made.

Credits for finding the issue go to James Bercegay of GulfTech Security
Research.

Regards,
Thijs Kinkhorst
SquirrelMail Development Team

------- Comment #2 From Sune Kloppenborg Jeppesen 2005-07-01 01:41:54 0000 -------
Created an attachment (id=62382) [details]
sqm_144_ident.diff

------- Comment #3 From Sune Kloppenborg Jeppesen 2005-07-01 01:44:36 0000 -------
Jeremy please advise. 
 
This seems rather hard to exploit. If you want some prerelease testing please 
attach an updated ebuild to this bug. Do NOT commit anything to Portage. 

------- Comment #4 From Jeremy Huddleston (RETIRED) 2005-07-02 10:21:38 0000 -------
Yeah... I tend to agree with Thijs that this is rather difficult to exploit. 
Combine that with the extensive nature of the changes here, and I'd prefer to
wait for upstream to finish testing with their 1.4.5 release with the fix next week.

------- Comment #5 From Sune Kloppenborg Jeppesen 2005-07-03 03:06:10 0000 -------
Ok, we'll wait on this one. 
 
Jeremy will you watch upstream for a new release? 

------- Comment #6 From Sune Kloppenborg Jeppesen 2005-07-11 07:55:05 0000 -------
1.4.5 will be released on Wednesday, we could just as well decide on GLSA 
publication already. I tend to vote NO. 

------- Comment #7 From Thierry Carrez (RETIRED) 2005-07-11 09:50:12 0000 -------
Yes I vote NO too.

------- Comment #8 From Thierry Carrez (RETIRED) 2005-07-13 12:49:29 0000 -------
Now public

------- Comment #9 From Thierry Carrez (RETIRED) 2005-07-13 12:50:38 0000 -------
*** Bug 98917 has been marked as a duplicate of this bug. ***

------- Comment #10 From Thierry Carrez (RETIRED) 2005-07-13 12:51:18 0000 -------
Jeremy : please bump to 1.4.5 final

------- Comment #11 From Jeremy Huddleston (RETIRED) 2005-07-13 13:50:58 0000 -------
in portage. ppc needs to mark stable.

------- Comment #12 From Jory A. Pratt 2005-07-13 14:00:12 0000 -------
Stable on PPC

------- Comment #13 From Sune Kloppenborg Jeppesen 2005-07-13 22:28:58 0000 -------
Reverting half NO to full NO -> Closing without GLSA. 
 
Thx everyone. 

------- Comment #14 From Thierry Carrez (RETIRED) 2005-07-14 02:07:58 0000 -------
and don't forget to close :)