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: Ifa59366dd400d3b36236c7a7ad83d3f3767b53a4
Signed-off-by: Martin Mucha <mmu...@redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/Range.java
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/92/29592/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..afc74aa 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);
}
--
To view, visit http://gerrit.ovirt.org/29592
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifa59366dd400d3b36236c7a7ad83d3f3767b53a4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Mucha <mmu...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
