From: Michael M Slusarz <slusarz@curecanti.org>
Date: Tue, 6 Oct 2009 22:13:10 +0000 (-0600)
Subject: Bug #8626: Fix updating memcache tracking info on session destroy
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=2eb27c9cb7d316e4665ac9fda78fa79836544427;p=horde.git

Bug #8626: Fix updating memcache tracking info on session destroy
---

diff --git a/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php b/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php
index f7209ff58..a82aecdbe 100644
--- a/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php
+++ b/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php
@@ -234,16 +234,15 @@ class Horde_SessionHandler_Memcache extends Horde_SessionHandler
         $result = $this->_memcache->delete($id);
         $this->_memcache->unlock($id);
 
-        if ($result !== false &&
-            isset($this->_persistent)) {
-            $result = $this->_persistent->destroy($id);
-        }
-
-        if ($result !== false) {
+        if ($result === false) {
             Horde::logMessage('Failed to delete session (id = ' . $id . ')', __FILE__, __LINE__, PEAR_LOG_DEBUG);
             return false;
         }
 
+        if (isset($this->_persistent)) {
+            $result = $this->_persistent->destroy($id);
+        }
+
         if (!empty($this->_params['track'])) {
             $this->_memcache->lock($this->_trackID);
             $ids = $this->_memcache->get($this->_trackID);