Fix potential nullpointer in recent KVM systemvm patchdisk update Signed-off-by: Edison Su <sudi...@gmail.com>
Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/f52d37fb Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/f52d37fb Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/f52d37fb Branch: refs/heads/javelin Commit: f52d37fb532179e3afc25cdae904553f5f96c8d1 Parents: 0d3a2a3 Author: Marcus Sorensen <shadow...@gmail.com> Authored: Fri Sep 14 15:26:29 2012 -0700 Committer: Edison Su <sudi...@gmail.com> Committed: Fri Sep 14 15:27:23 2012 -0700 ---------------------------------------------------------------------- .../kvm/resource/LibvirtComputingResource.java | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/f52d37fb/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java ---------------------------------------------------------------------- diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java index 9312519..65742d8 100755 --- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java +++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java @@ -2587,7 +2587,8 @@ public class LibvirtComputingResource extends ServerResourceBase implements if (disk.getDeviceType() == DiskDef.deviceType.CDROM && disk.getDiskPath() != null) { cleanupDisk(conn, disk); - } else if (disk.getDiskPath().contains(vmName + "-patchdisk") + } else if (disk.getDiskPath() != null + && disk.getDiskPath().contains(vmName + "-patchdisk") && vmName.matches("^[rsv]-\\d+-VM$")) { if (!_storagePoolMgr.deleteVbdByPath(disk.getDiskPath())) { s_logger.warn("failed to delete patch disk " + disk.getDiskPath());