summaryrefslogtreecommitdiffstats
path: root/lib/private/Lock/MemcacheLockingProvider.php
diff options
context:
space:
mode:
Diffstat (limited to 'lib/private/Lock/MemcacheLockingProvider.php')
-rw-r--r--lib/private/Lock/MemcacheLockingProvider.php6
1 files changed, 5 insertions, 1 deletions
diff --git a/lib/private/Lock/MemcacheLockingProvider.php b/lib/private/Lock/MemcacheLockingProvider.php
index 3d7510f43ff..bff4eabbbd8 100644
--- a/lib/private/Lock/MemcacheLockingProvider.php
+++ b/lib/private/Lock/MemcacheLockingProvider.php
@@ -94,8 +94,12 @@ class MemcacheLockingProvider extends AbstractLockingProvider {
*/
public function releaseLock(string $path, int $type) {
if ($type === self::LOCK_SHARED) {
+ $ownSharedLockCount = $this->getOwnSharedLockCount($path);
$newValue = 0;
- if ($this->getOwnSharedLockCount($path) === 1) {
+ if ($ownSharedLockCount === 0) { // if we are not holding the lock, don't try to release it
+ return;
+ }
+ if ($ownSharedLockCount === 1) {
$removed = $this->memcache->cad($path, 1); // if we're the only one having a shared lock we can remove it in one go
if (!$removed) { //someone else also has a shared lock, decrease only
$newValue = $this->memcache->dec($path);