diff options
author | Andrey Andreev <narf@devilix.net> | 2017-07-06 10:49:13 +0200 |
---|---|---|
committer | Andrey Andreev <narf@devilix.net> | 2017-07-06 10:50:22 +0200 |
commit | fdf4b59e7734e3417c745a651382dec8bf556ed0 (patch) | |
tree | eb0e1b46d206b9f3c59ccef2e88f8aff1e898c9e /system/libraries/Session/drivers/Session_redis_driver.php | |
parent | 59bae57503865f6dcea91fc20fd2729b2e79bb74 (diff) |
[ci skip] Merge pull request #5170 from tianhe1986/develop_session_race_condition
Decreasing the probability of race condition in session lock
Diffstat (limited to 'system/libraries/Session/drivers/Session_redis_driver.php')
-rw-r--r-- | system/libraries/Session/drivers/Session_redis_driver.php | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/system/libraries/Session/drivers/Session_redis_driver.php b/system/libraries/Session/drivers/Session_redis_driver.php index e220a2951..a9e655a8c 100644 --- a/system/libraries/Session/drivers/Session_redis_driver.php +++ b/system/libraries/Session/drivers/Session_redis_driver.php @@ -341,7 +341,11 @@ class CI_Session_redis_driver extends CI_Session_driver implements SessionHandle continue; } - if ( ! $this->_redis->setex($lock_key, 300, time())) + $result = ($ttl === -2) + ? $this->_redis->set($lock_key, time(), array('nx', 'ex' => 300)) + : $this->_redis->setex($lock_key, 300, time()); + + if ( ! $result) { log_message('error', 'Session: Error while trying to obtain lock for '.$this->_key_prefix.$session_id); return FALSE; |