Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix FB on AddExistingFileStorageDomain ......................................................................
core: Fix FB on AddExistingFileStorageDomain This patch reverts 5ccea20c205457f70cbd480f80455f600b386836 and offers a proper fix for the (bogus) NP_NULL_PARAM_DEREF error in AddExistingFileStorageDomainCommand. The domainFromIrs variable can NEVER be null, hence it's meaningless to protect against it. Change-Id: Id8c5a11ef872cac6aeebfa18bf28757fb4a8b26c Signed-off-by: Allon Mureinik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/AddExistingFileStorageDomainCommand.java 1 file changed, 11 insertions(+), 15 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22331 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id8c5a11ef872cac6aeebfa18bf28757fb4a8b26c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Sergey Gotliv <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
