Martin Mucha has uploaded a new change for review. Change subject: core: error fix: right bound of range interval should be inclusive. ......................................................................
core: error fix: right bound of range interval should be inclusive. number of macs in range was invalid, missing one mac, which causes system complaining about no available mac, when same start mac and end mac was used. This is not ok, since validations allows use of same start mac and end mac for given mac range. Change-Id: I7f1257e752680b56decd69eb9a79caad97378e12 Bug-Url: https://bugzilla.redhat.com/1127245 Signed-off-by: Martin Mucha <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/Range.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/RangeTest.java 2 files changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/16/31316/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/Range.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/Range.java index b519576..3a7dea5 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/Range.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/Range.java @@ -20,7 +20,7 @@ this.rangeStart = rangeStart; this.rangeEnd = rangeEnd; - int numberOfMacs = (int) (rangeEnd - rangeStart); + int numberOfMacs = (int) (rangeEnd - rangeStart) + 1; this.availableMacsCount = numberOfMacs; this.usedMacs = new BitSet(numberOfMacs); } diff --git a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/RangeTest.java b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/RangeTest.java index 333d1cf..77eb526 100644 --- a/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/RangeTest.java +++ b/backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/RangeTest.java @@ -13,7 +13,7 @@ private static final int MAC_FROM_RANGE = 15; private static final int MAC_OUTSIDE_OF_RANGE = 25; private static final int RANGE_FROM = 10; - private static final int RANGE_TO = 20; + private static final int RANGE_TO = 19; private static final int NUMBER_OF_MACS = 10; private Range rangeOf10Macs; -- To view, visit http://gerrit.ovirt.org/31316 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7f1257e752680b56decd69eb9a79caad97378e12 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Martin Mucha <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
