Tal Nisan has submitted this change and it was merged. 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(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11833 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica20a2542c07a355720503c25a14fbae3394c4d4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
