From 44021155b0b48c88878b2e0a0f7cd6ef54de5156 Mon Sep 17 00:00:00 2001
From: "Jeroen van Meeuwen (Kolab Systems)" <vanmeeuwen@kolabsys.com>
Date: Mon, 19 Aug 2013 13:54:08 +0200
Subject: [PATCH] Make sure we loop over non-empty arrays only

---
 plugins/kolab_auth/kolab_auth.php | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/plugins/kolab_auth/kolab_auth.php b/plugins/kolab_auth/kolab_auth.php
index 27c7b563..b139e327 100644
--- a/plugins/kolab_auth/kolab_auth.php
+++ b/plugins/kolab_auth/kolab_auth.php
@@ -114,12 +114,16 @@ class kolab_auth extends rcube_plugin
             return $args;
         }
 
-        foreach ($role_plugins as $role_dn => $plugins) {
-            $role_plugins[$ldap->parse_vars($role_dn)] = $plugins;
+        if (!empty($role_plugins)) {
+            foreach ($role_plugins as $role_dn => $plugins) {
+                $role_plugins[$ldap->parse_vars($role_dn)] = $plugins;
+            }
         }
 
-        foreach ($role_settings as $role_dn => $settings) {
-            $role_settings[$ldap->parse_vars($role_dn)] = $settings;
+        if (!empty($role_settings)) {
+            foreach ($role_settings as $role_dn => $settings) {
+                $role_settings[$ldap->parse_vars($role_dn)] = $settings;
+            }
         }
 
         foreach ($role_dns as $role_dn) {