#118 Fatal error: Only variables can be passed by reference

[Anonymous]

* Fatal error: Only variables can be passed by
reference in
/usr/local/apache2/htdocs/squirrelmail/functions/imap_messages.php
on line 480

After upgrading from PHP4 to PHP5.1, my Squirrelmail
now sends this message login, when I click on INBOX, or
when I click on "sent." Everything else continues to work.

[Jonathan Angliss]

Logged In: YES
user_id=620333

We need to know what version of SquirrelMail you are using.

[Tomas Kuliavas]

Logged In: YES
user_id=225877

Please provide more details about used squirrelmail version
or Id tag of imap_messages.php file

try reproducing your error in 1.5.1cvs or 1.4.5rc1

[Tomas Kuliavas]

[Tomas Kuliavas]

Logged In: YES
user_id=225877

it is enough to divide split() and array_shift() into two
calls.

[Nobody/Anonymous]

Logged In: NO

I am using Squirrelmail version 1.4.5. I tried installing
the CVS, but I had the notorious "&This" error.

Thanks,

Mike

[Nobody/Anonymous]

Logged In: NO

Hi Tokul,

Thanks so much.

Mike

[Nobody/Anonymous]

Logged In: NO

But now when I try to open the message (now that I can see
the messages in the inbox), I get the following error:

Fatal error: Only variables can be passed by reference in
/usr/local/apache2/htdocs/squirrelmail/class/mime/Rfc822Header.class.php
on line 509

Mike

[Tomas Kuliavas]

Logged In: YES
user_id=225877

same issue.

SquirrelMail 1.4.5 uses array_shift() function with function
all as argument in four places.

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

Attached patch for all four calls + some small fixes in
src/configtest.php and src/search.php.

It should be fixed in 1.4.6cvs, but we'll try fixing devel
first.

[Tomas Kuliavas]

second patch. I think all array_shift()s are fixed

[Nobody/Anonymous]

Logged In: NO

Tokul,

Thanks once again. I really appreciate it.

Mike

[Tomas Kuliavas]

Logged In: YES
user_id=225877

Fixed in 1.5.1cvs and 1.4.6cvs

$this error in functions/imap_mailbox.php still happens in
devel branch, but we have other bug report (#1190160) for it.