Vered Volansky has uploaded a new change for review. Change subject: core: Avoid NPE when changing image status (868887) ......................................................................
core: Avoid NPE when changing image status (868887) Sometimes BaseImagesCommand.setImageStatus() checked for getRelevantDiskImage() without checking for null. Higher up in the hierarchy the getDiskImage returned value is tested for null (before unlockImage() calls), even though the status change is always done on getRelevantDiskImage() returned value. Now what is tested for null before unlockImage() is getRelevantDiskImage(). Change-Id: Ica20a2542c07a355720503c25a14fbae3394c4d4 Bug-Url: https://bugzilla.redhat.com/868887 Signed-off-by: Vered Volansky <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java 1 file changed, 2 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/33/11833/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java index 4891254..39251d4 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java @@ -385,8 +385,7 @@ getImageDao().update(getDestinationDiskImage().getImage()); } - if (getDiskImage() != null) { - // Unlock source image: + if (getRelevantDiskImage() != null) { unLockImage(); } @@ -405,8 +404,7 @@ removeSnapshot(getDestinationDiskImage()); } - if (getDiskImage() != null) { - // Unlock source image: + if (getRelevantDiskImage() != null) { unLockImage(); } } -- To view, visit http://gerrit.ovirt.org/11833 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ica20a2542c07a355720503c25a14fbae3394c4d4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
