| commit | 20ded699347d4b54dae3086bfa20ed3cb5e5a3e1 | [log] [tgz] |
|---|---|---|
| author | Pavlin Radoslavov <pavlin@onlab.us> | Mon Nov 17 16:03:15 2014 -0800 |
| committer | Gerrit Code Review <gerrit@onlab.us> | Mon Nov 17 16:38:15 2014 -0800 |
| tree | 4f34e9872cf2c5a3ca25fb94dc585fb8a3365b7e | |
| parent | 269670fc8ca9799d1dce2560f38ce9b3506e298e [diff] |
Fix a bug inside DistributedLock.extendExpiration(): Return false (as per API) if we don't hold the lock. Change-Id: Id3c314f6e374c92a73ce6f1aa4b6fbcef0a80697
diff --git a/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/DistributedLock.java b/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/DistributedLock.java index a3fbc0d..48a45c4 100644 --- a/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/DistributedLock.java +++ b/core/store/dist/src/main/java/org/onlab/onos/store/service/impl/DistributedLock.java
@@ -132,6 +132,7 @@ if (!isLocked()) { log.warn("Ignoring request to extend expiration for lock {}." + " ExtendExpiration must be called for locks that are already acquired.", path); + return false; } if (databaseService.putIfValueMatches(