From 6b51cbd98cfc45f90fd4b61ae9ea28c23999288f Mon Sep 17 00:00:00 2001 From: Michal Cmarada Date: Thu, 3 May 2018 09:42:34 +0200 Subject: HC2VPP-314 - fix for IPV6 routes with 128 subnet prefix VPP uses byte in range of 0 to 255 but when jni is used and this type is converted to java byte which has range of -128 to 127 it will convert it to negative value. It then fails to create Ip6Prefix from this value. Change-Id: Ic18686959682c153da2e4ee4a7f7841c9b56e5d3 Signed-off-by: Michal Cmarada --- .../main/java/io/fd/hc2vpp/routing/naming/Ipv6RouteNamesFactory.java | 2 +- .../java/io/fd/hc2vpp/routing/write/trait/RouteRequestProducer.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'routing/routing-impl/src/main/java/io/fd') diff --git a/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/naming/Ipv6RouteNamesFactory.java b/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/naming/Ipv6RouteNamesFactory.java index a58051b63..abb48618c 100644 --- a/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/naming/Ipv6RouteNamesFactory.java +++ b/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/naming/Ipv6RouteNamesFactory.java @@ -60,7 +60,7 @@ public final class Ipv6RouteNamesFactory implements RouteMapper { public String uniqueRouteName(@Nonnull final Ip6FibDetails details, @Nonnull final MappingContext mappingContext) { return bindName(routingProtocolContext.getName(details.tableId, mappingContext), doubleDotlessAddress(details.address), - String.valueOf(details.addressLength)); + String.valueOf(Byte.toUnsignedInt(details.addressLength))); } public String uniqueRouteHopName( diff --git a/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/write/trait/RouteRequestProducer.java b/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/write/trait/RouteRequestProducer.java index 80b4b8f79..97adb3589 100644 --- a/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/write/trait/RouteRequestProducer.java +++ b/routing/routing-impl/src/main/java/io/fd/hc2vpp/routing/write/trait/RouteRequestProducer.java @@ -63,7 +63,7 @@ public interface RouteRequestProducer extends ByteDataTranslator, AddressTransla checkArgument( allowedPrefixPatterns.stream().anyMatch(pattern -> Pattern.compile(pattern).matcher(value).matches()), "%s is not a valid Ip-prefix value"); - return Byte.valueOf(value.substring(value.indexOf("/") + 1)); + return Integer.valueOf(value.substring(value.indexOf("/") + 1)).byteValue(); } default boolean classifyTablePresent(final String classifyTableName, -- cgit 1.2.3-korg