Fri, 10 Apr 2026 13:16:46 +0000 https://bugs.horde.org/ticket/14580 Gollem Auth thorws warning and pollutes Errorlog - fix available Gollem Auth (gollem/lib/Auth.php ~line 340) throws warnings: "PHP ERROR: Invalid argument supplied for foreach()" Since the causing function _getBackends() is invoked several times, it pollutes the logfile. In the mentioned function, the passwords are retrieved and iterated via foreach. However, if only a singlr password is returned, PHP complains that it is no array. I propose to insert an cast into array to have a defined type presented to foreach() to correct the problem. The fixed function should be like this: gollem/lib/Auth.php: /** * Return stored backend list. * * @return array Backend configuration list. */ protected static function _getBackends() { global $session; if (($backends = $session->get('gollem', 'backends', $session::TYPE_ARRAY)) && ($passwords = $session->get('gollem', 'backends_password', $session::TYPE_ARRAY))) { foreach ( (array) $passwords as $key => $val) { $backends[$key]['params']['password'] = $val; } } return $backends; } Thanks, Torben Thu, 16 Feb 2017 16:01:27 +0000 https://bugs.horde.org/ticket/14580#t91597 The real question is why is a non-array value returned from $session->get when the TYPE_ARRAY mask is passed? Perhaps it's an issue in Horde_Session when a value is saved with the ENCRYPT mask but retrieved with the TYPE_ARRAY mask. Fri, 24 Feb 2017 17:55:32 +0000 https://bugs.horde.org/ticket/14580#t91634 There seems to be some issue with decompressing certain arrays with numeric keys. I've been able to reproduce it on one machine using PHP 5.6, but not on my dev machine running php 7. Still investigating. Thu, 02 Mar 2017 17:03:25 +0000 https://bugs.horde.org/ticket/14580#t91671 > There seems to be some issue with decompressing certain arrays with > numeric keys. I've been able to reproduce it on one machine using PHP > 5.6, but not on my dev machine running php 7. > > Still investigating. Thanks for the update Thu, 02 Mar 2017 17:31:20 +0000 https://bugs.horde.org/ticket/14580#t91672 > There seems to be some issue with decompressing certain arrays with > numeric keys. I've been able to reproduce it on one machine using PHP > 5.6, but not on my dev machine running php 7. > > Still investigating. ....and now I can't reproduce it at all :( Thu, 02 Mar 2017 17:40:33 +0000 https://bugs.horde.org/ticket/14580#t91673 Can you see what $passwords is equal to right before the for loop? When it did break for me, it was an unencrypted binary string when the original $password array contained a single entry in element (int)0. If the entry was "moved" to element '1', it worked but I can't get it to break again. It's always returned as an array now. Thu, 02 Mar 2017 17:48:43 +0000 https://bugs.horde.org/ticket/14580#t91674 > Can you see what $passwords is equal to right before the for loop? > > When it did break for me, it was an unencrypted binary string when > the original $password array contained a single entry in element > (int)0. If the entry was "moved" to element '1', it worked but I > can't get it to break again. It's always returned as an array now. I'll check that, but I'm on the road for four days. I'll come back to this issue on Tuesday or Wednesday. Thu, 02 Mar 2017 21:35:12 +0000 https://bugs.horde.org/ticket/14580#t91675 >> Can you see what $passwords is equal to right before the for loop? >> >> When it did break for me, it was an unencrypted binary string when >> the original $password array contained a single entry in element >> (int)0. If the entry was "moved" to element '1', it worked but I >> can't get it to break again. It's always returned as an array now. > > I'll check that, but I'm on the road for four days. I'll come back to > this issue on Tuesday or Wednesday. Mike, using the current set of stable pear packages of Horde, I'm also not able to reproduce it. I recommend to close the bug. Thanks for your help. Mon, 03 Apr 2017 13:24:39 +0000 https://bugs.horde.org/ticket/14580#t91754