Repository: cloudstack Updated Branches: refs/heads/master bfa36c028 -> b1d0df16c
CID-1116258: Guard _storagePools using synchronized in KVMStoragePoolManager Signed-off-by: Rohit Yadav <rohit.ya...@shapeblue.com> Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b1d0df16 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b1d0df16 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b1d0df16 Branch: refs/heads/master Commit: b1d0df16cf26ffa0f7b077c5966e50c8ede1ef98 Parents: bfa36c0 Author: Rohit Yadav <rohit.ya...@shapeblue.com> Authored: Thu Sep 18 11:48:11 2014 +0200 Committer: Rohit Yadav <rohit.ya...@shapeblue.com> Committed: Thu Sep 18 11:48:11 2014 +0200 ---------------------------------------------------------------------- .../com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/b1d0df16/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java ---------------------------------------------------------------------- diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java index f44bb03..ab819b2 100644 --- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java +++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStoragePoolManager.java @@ -300,7 +300,9 @@ public class KVMStoragePoolManager { StorageAdaptor adaptor = getStorageAdaptor(type); _haMonitor.removeStoragePool(uuid); adaptor.deleteStoragePool(uuid); - _storagePools.remove(uuid); + synchronized (_storagePools) { + _storagePools.remove(uuid); + } return true; }
