Oved Ourfali has uploaded a new change for review. Change subject: core: fix split function and its use for spice security channels ......................................................................
core: fix split function and its use for spice security channels Previous commit added the split function, but it didn't take into account what if the entries are not there at all (new install). It assumed they are there, as they would appear in the add section. Better implementation is not to assume that, creating the old entries with old value in install phase, or taking the original value from the DB in case of upgrade. Change-Id: I32b9b89c0c28180738b8532561e110fcd3e61d1b Signed-off-by: Oved Ourfali <oourf...@redhat.com> --- M backend/manager/dbscripts/common_sp.sql M backend/manager/dbscripts/upgrade/pre_upgrade/config.sql 2 files changed, 16 insertions(+), 10 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/71/4271/1 -- To view, visit http://gerrit.ovirt.org/4271 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I32b9b89c0c28180738b8532561e110fcd3e61d1b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourf...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches