From 9ba230615770e67b3d8877d73efac28757696e0a Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Fri, 19 Feb 2010 11:26:26 -0800
Subject: [PATCH] Properly return groups for the guest user.

---
 modules/ldap/libraries/drivers/IdentityProvider/Ldap.php | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php b/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php
index 6eabc102..139eadf4 100644
--- a/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php
+++ b/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php
@@ -178,13 +178,16 @@ class IdentityProvider_Ldap_Driver implements IdentityProvider_Driver {
   public function groups() {
     $groups = array();
     foreach (self::$_params["groups"] as $group_name) {
-      $root = item::root();
       $groups[] = $this->lookup_group_by_name($group_name);
     }
     return $groups;
   }
 
   static function groups_for($user) {
+    if ($user->guest) {
+      return $user->groups;
+    }
+
     $result = ldap_search(self::$_connection, self::$_params["group_domain"],
                           "(memberUid=$user->name)");