|
Lines 1-91
Link Here
|
| 1 |
--- program/include/rcube_session.php.orig 2012-08-06 18:18:13.000000000 +0200 |
|
|
| 2 |
+++ program/include/rcube_session.php 2012-08-13 12:44:54.000000000 +0200 |
| 3 |
@@ -37,7 +37,6 @@ |
| 4 |
private $ip; |
| 5 |
private $start; |
| 6 |
private $changed; |
| 7 |
- private $unsets = array(); |
| 8 |
private $gc_handlers = array(); |
| 9 |
private $cookiename = 'roundcube_sessauth'; |
| 10 |
private $vars; |
| 11 |
@@ -134,7 +133,7 @@ |
| 12 |
if ($sql_result && ($sql_arr = $this->db->fetch_assoc($sql_result))) { |
| 13 |
$this->changed = strtotime($sql_arr['changed']); |
| 14 |
$this->ip = $sql_arr['ip']; |
| 15 |
- $this->vars = base64_decode($sql_arr['vars']); |
| 16 |
+ $this->vars = $sql_arr['vars']; |
| 17 |
$this->key = $key; |
| 18 |
|
| 19 |
return !empty($this->vars) ? (string) $this->vars : ''; |
| 20 |
@@ -170,13 +169,13 @@ |
| 21 |
} |
| 22 |
|
| 23 |
if ($oldvars !== null) { |
| 24 |
- $newvars = $this->_fixvars($vars, $oldvars); |
| 25 |
+ $newvars = $vars; |
| 26 |
|
| 27 |
if ($newvars !== $oldvars) { |
| 28 |
$this->db->query( |
| 29 |
sprintf("UPDATE %s SET vars=?, changed=%s WHERE sess_id=?", |
| 30 |
get_table_name('session'), $now), |
| 31 |
- base64_encode($newvars), $key); |
| 32 |
+ $newvars, $key); |
| 33 |
} |
| 34 |
else if ($ts - $this->changed > $this->lifetime / 2) { |
| 35 |
$this->db->query("UPDATE ".get_table_name('session')." SET changed=$now WHERE sess_id=?", $key); |
| 36 |
@@ -187,7 +186,7 @@ |
| 37 |
sprintf("INSERT INTO %s (sess_id, vars, ip, created, changed) ". |
| 38 |
"VALUES (?, ?, ?, %s, %s)", |
| 39 |
get_table_name('session'), $now, $now), |
| 40 |
- $key, base64_encode($vars), (string)$this->ip); |
| 41 |
+ $key, $vars, (string)$this->ip); |
| 42 |
} |
| 43 |
|
| 44 |
return true; |
| 45 |
@@ -195,29 +194,6 @@ |
| 46 |
|
| 47 |
|
| 48 |
/** |
| 49 |
- * Merge vars with old vars and apply unsets |
| 50 |
- */ |
| 51 |
- private function _fixvars($vars, $oldvars) |
| 52 |
- { |
| 53 |
- if ($oldvars !== null) { |
| 54 |
- $a_oldvars = $this->unserialize($oldvars); |
| 55 |
- if (is_array($a_oldvars)) { |
| 56 |
- foreach ((array)$this->unsets as $k) |
| 57 |
- unset($a_oldvars[$k]); |
| 58 |
- |
| 59 |
- $newvars = $this->serialize(array_merge( |
| 60 |
- (array)$a_oldvars, (array)$this->unserialize($vars))); |
| 61 |
- } |
| 62 |
- else |
| 63 |
- $newvars = $vars; |
| 64 |
- } |
| 65 |
- |
| 66 |
- $this->unsets = array(); |
| 67 |
- return $newvars; |
| 68 |
- } |
| 69 |
- |
| 70 |
- |
| 71 |
- /** |
| 72 |
* Handler for session_destroy() |
| 73 |
* |
| 74 |
* @param string Session ID |
| 75 |
@@ -296,7 +272,7 @@ |
| 76 |
else // else read data again |
| 77 |
$oldvars = $this->mc_read($key); |
| 78 |
|
| 79 |
- $newvars = $oldvars !== null ? $this->_fixvars($vars, $oldvars) : $vars; |
| 80 |
+ $newvars = $vars; |
| 81 |
|
| 82 |
if ($newvars !== $oldvars || $ts - $this->changed > $this->lifetime / 2) |
| 83 |
return $this->memcache->set($key, serialize(array('changed' => time(), 'ip' => $this->ip, 'vars' => $newvars)), MEMCACHE_COMPRESSED, $this->lifetime); |
| 84 |
@@ -376,7 +352,6 @@ |
| 85 |
if (empty($var)) |
| 86 |
return $this->destroy(session_id()); |
| 87 |
|
| 88 |
- $this->unsets[] = $var; |
| 89 |
unset($_SESSION[$var]); |
| 90 |
|
| 91 |
return true; |