From Arik Hadas <ahadas@redhat.com>:

Arik Hadas has submitted this change and it was merged. ( https://gerrit.ovirt.org/79258 ) Change subject: engine: handle NPE for guest info data of a powered-off vm ...................................................................... engine: handle NPE for guest info data of a powered-off vm In case of powering-off a running VM with guest info data (i.e. guest agent installed) and guestOsType == OsType.Windows, then displaying 'guest info' sub-tab raises a NPE. This is becuase powering off a VM, sets vm.guestOs field to null in ResourceManager.resetVmAttributes(), while other Os fields like guestOsType are not set to null. That causes VmGuestInfoModel.setGuestOsNamedVersion() to fail while trying to access that field. This patch fixes it by avoid setting vm.guestOs to null. Bug-Url: https://bugzilla.redhat.com/1448650 Change-Id: I4f84fbe6abbc74bae54de80d8f5348646b8df3b4 Signed-off-by: Sharon Gratch <sgratch@redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResourceManager.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGuestInfoModel.java 2 files changed, 1 insertion(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Sharon Gratch: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/79258 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f84fbe6abbc74bae54de80d8f5348646b8df3b4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharon Gratch <sgratch@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Gratch <sgratch@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>