Bug 108394 – squirrelmail 1.4.5 uses array_shift in a wrong way (imap_messages.php)

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Bjarke Istrup Pedersen (RETIRED) 2005-10-07 08:34:16 UTC]

There is a bug in the squirrelmail packages that causes it to not wanting to 
show the users emails.
There is a patch in CVS on sourceforge.

Reproducible: Always
Steps to Reproduce:
1. emerge squirrelmail
2. install it with webapp-config and configure it
3. check your inbox

Actual Results:  
It doesn't show my emails, but an error.

Expected Results:  
It should have showed my emails, after applying the patch it works.

[Comment 1 Bjarke Istrup Pedersen (RETIRED) 2005-10-07 08:35:04 UTC]

Created attachment 70094 [details, diff]
Patch that fixes the bug

This patch fixes the bug.

[Comment 2 Bjarke Istrup Pedersen (RETIRED) 2005-10-15 15:18:20 UTC]

Created attachment 70752 [details, diff]
Patch that fixes both bugs

This patch fixes another bug, along with the first bug, the error is really the
same problems, just in two different files.

Here is the error from the second bug:
Fatal error: Only variables can be passed by reference in
squirrelmail/class/mime/Rfc822Header.class.php on line 509

[Comment 3 Jeremy Huddleston (RETIRED) 2005-10-15 16:07:28 UTC]

fixed in portage.  thanks.

[Comment 4 MAL 2006-01-28 02:18:20 UTC]

Can we get this stable please?

[Comment 5 Philippe Trottier (RETIRED) 2006-02-06 01:22:40 UTC]

There is 4 places with this issue, not only 2, and the patch is actually bigger

src/configtest.php
plugins/listcommands/setup.php
class/mime/Rfc822Header.php
functions/imap_messages.php

I can be marked as the ppc tester for this, as I use it for my company and recently upgraded to php5. You could also put me here as a developer and fix the issues, adding me to the herd, whatever.

http://sourceforge.net/tracker/index.php?func=detail&aid=1237160&group_id=311&atid=423679

[Comment 6 Philippe Trottier (RETIRED) 2006-02-06 01:25:42 UTC]

It is a blocker for those who install it for the 1st time, not for those who who already have it installed.

[Comment 7 Jeremy Huddleston (RETIRED) 2006-03-31 22:01:31 UTC]

this is fixed in the versions in portage.