Tickets :: Comment on [#14523] Invalid argument supplied for foreach() (gollem/lib/Auth.php)

* Your Email Address
* Spam protection	Enter the letters below: .__.. ._..__..___. \| \|\| \| \| \| _/ \|__\\|____\|_\|__\./__.
Comment	> I get these once in a while in syslog, under a Horde-5.2.6 webmail > installation. > > [gollem] PHP ERROR: Invalid argument supplied for foreach() [pid 8891 > on line 338 of "?/gollem/lib/Auth.php"] > > Tracing the behaviour, it appears the following code line 336: > > $passwords = $session->get('gollem', 'backends_password', > $session::ENCRYPT); > > does not always assign an array, sometimes it assigns a binary string > (related to the json_encode() behavior described in bug #14522 > perhaps?): > > I'd hence suggest, if this binary string behavior is to be expected, > to change line 337 from: > > if ($passwords) { > > to: > > if (is_array($passwords)) { > > so that this syslog message is silenced. > > Thanks!
Attachment
Watch this ticket