summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichal Cmarada <mcmarada@cisco.com>2019-06-04 17:44:32 +0200
committerMichal Cmarada <mcmarada@cisco.com>2019-06-04 17:44:32 +0200
commite2746559f42636e667513024c85c9a6a20de0634 (patch)
tree3f16fda0f87d53d5c84ac117f67da58a2b93c7d9
parent27c59c4083cb300f528d63b3e3924c348dd090e0 (diff)
fix sonar issues
- fixes issues for wrong modifier order - fixes wrong @Nonnull annotations in AclReader - fixes unnecessary boxing to Long Change-Id: I5713d81f24d21cbefcc5cf3398e75c14e0526f6c Signed-off-by: Michal Cmarada <mcmarada@cisco.com>
-rw-r--r--acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/AclIIds.java14
-rw-r--r--ipsec/ipsec-impl/src/main/java/io/fd/hc2vpp/ipsec/read/IpsecStateCustomizer.java9
-rw-r--r--srv6/srv6-impl/src/main/java/io/fd/hc2vpp/srv6/Srv6IIds.java68
-rw-r--r--vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/read/acl/AclReader.java3
4 files changed, 43 insertions, 51 deletions
diff --git a/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/AclIIds.java b/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/AclIIds.java
index 7ed2fe6a5..4ea5ad3b3 100644
--- a/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/AclIIds.java
+++ b/acl/acl-impl/src/main/java/io/fd/hc2vpp/acl/AclIIds.java
@@ -51,20 +51,20 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.
import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
public class AclIIds {
- public final static InstanceIdentifier<Acls> ACLS = InstanceIdentifier.create(Acls.class);
- public final static InstanceIdentifier<Acl> ACLS_ACL = ACLS.child(Acl.class);
- public final static InstanceIdentifier<Acl> ACL = InstanceIdentifier.create(Acl.class);
+ public static final InstanceIdentifier<Acls> ACLS = InstanceIdentifier.create(Acls.class);
+ public static final InstanceIdentifier<Acl> ACLS_ACL = ACLS.child(Acl.class);
+ public static final InstanceIdentifier<Acl> ACL = InstanceIdentifier.create(Acl.class);
- public final static InstanceIdentifier<AttachmentPoints> ACLS_AP = ACLS.child(AttachmentPoints.class);
- public final static InstanceIdentifier<Interface> ACLS_AP_INT = ACLS_AP.child(Interface.class);
+ public static final InstanceIdentifier<AttachmentPoints> ACLS_AP = ACLS.child(AttachmentPoints.class);
+ public static final InstanceIdentifier<Interface> ACLS_AP_INT = ACLS_AP.child(Interface.class);
public static final InstanceIdentifier<Ingress> ACLS_AP_INT_ING = ACLS_AP_INT.child(Ingress.class);
public static final InstanceIdentifier<AclSets> ACLS_AP_INT_ING_ACLS = ACLS_AP_INT_ING.child(AclSets.class);
public static final InstanceIdentifier<AclSet> ACLS_AP_INT_ING_ACLS_ACL = ACLS_AP_INT_ING_ACLS.child(AclSet.class);
public static final InstanceIdentifier<Egress> ACLS_AP_INT_EGR = ACLS_AP_INT.child(Egress.class);
public static final InstanceIdentifier<AclSets> ACLS_AP_INT_EGR_ACLS = ACLS_AP_INT_EGR.child(AclSets.class);
public static final InstanceIdentifier<AclSet> ACLS_AP_INT_EGR_ACLS_ACL = ACLS_AP_INT_EGR_ACLS.child(AclSet.class);
- public final static InstanceIdentifier<Interface> IFC_ACL = InstanceIdentifier.create(Interface.class);
- public final static InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev180220.interfaces.Interface>
+ public static final InstanceIdentifier<Interface> IFC_ACL = InstanceIdentifier.create(Interface.class);
+ public static final InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev180220.interfaces.Interface>
IFC = InstanceIdentifier.create(Interfaces.class)
.child(org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev180220.interfaces.Interface.class);
public static final InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev180220.interfaces.state.Interface>
diff --git a/ipsec/ipsec-impl/src/main/java/io/fd/hc2vpp/ipsec/read/IpsecStateCustomizer.java b/ipsec/ipsec-impl/src/main/java/io/fd/hc2vpp/ipsec/read/IpsecStateCustomizer.java
index 3148d144b..2049e6144 100644
--- a/ipsec/ipsec-impl/src/main/java/io/fd/hc2vpp/ipsec/read/IpsecStateCustomizer.java
+++ b/ipsec/ipsec-impl/src/main/java/io/fd/hc2vpp/ipsec/read/IpsecStateCustomizer.java
@@ -87,7 +87,7 @@ public class IpsecStateCustomizer extends FutureJVppCustomizer
for (IpsecSaDetails details : reply.ipsecSaDetails) {
SaBuilder saBuilder = new SaBuilder();
saBuilder.setSpi(Integer.toUnsignedLong(details.entry.spi))
- .setAntiReplayWindow(Long.valueOf(details.replayWindow).intValue())
+ .setAntiReplayWindow((int) details.replayWindow)
.setAuthenticationAlgorithm(parseAuthAlgorithm(details.entry.integrityAlgorithm))
.setEncryptionAlgorithm(parseCryptoAlgorithm(details.entry.cryptoAlgorithm));
listSa.add(saBuilder.build());
@@ -106,14 +106,10 @@ public class IpsecStateCustomizer extends FutureJVppCustomizer
return IkeEncryptionAlgorithmT.EncrAesCbc192;
case IPSEC_API_CRYPTO_ALG_AES_CBC_256:
return IkeEncryptionAlgorithmT.EncrAesCbc256;
+ // todo verify Cryptoalgorithms
case IPSEC_API_CRYPTO_ALG_AES_CTR_128:
- // todo verify Cryptoalgorithms
- return IkeEncryptionAlgorithmT.EncrAesCtr;
case IPSEC_API_CRYPTO_ALG_AES_CTR_192:
- // todo verify Cryptoalgorithms
- return IkeEncryptionAlgorithmT.EncrAesCtr;
case IPSEC_API_CRYPTO_ALG_AES_CTR_256:
- // todo verify Cryptoalgorithms
return IkeEncryptionAlgorithmT.EncrAesCtr;
case IPSEC_API_CRYPTO_ALG_AES_GCM_128:
return IkeEncryptionAlgorithmT.EncrAesGcm8Icv;
@@ -122,7 +118,6 @@ public class IpsecStateCustomizer extends FutureJVppCustomizer
case IPSEC_API_CRYPTO_ALG_AES_GCM_256:
return IkeEncryptionAlgorithmT.EncrAesGcm16Icv;
case IPSEC_API_CRYPTO_ALG_DES_CBC:
- // todo verify Cryptoalgorithms
return IkeEncryptionAlgorithmT.EncrDes;
case IPSEC_API_CRYPTO_ALG_3DES_CBC:
return IkeEncryptionAlgorithmT.Encr3des;
diff --git a/srv6/srv6-impl/src/main/java/io/fd/hc2vpp/srv6/Srv6IIds.java b/srv6/srv6-impl/src/main/java/io/fd/hc2vpp/srv6/Srv6IIds.java
index 3097ba356..8450afb5e 100644
--- a/srv6/srv6-impl/src/main/java/io/fd/hc2vpp/srv6/Srv6IIds.java
+++ b/srv6/srv6-impl/src/main/java/io/fd/hc2vpp/srv6/Srv6IIds.java
@@ -47,50 +47,50 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6.base.r
import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
public class Srv6IIds {
- public final static InstanceIdentifier<Routing> RT = InstanceIdentifier.create(Routing.class);
- public final static InstanceIdentifier<Routing1> RT_RT1_AUG = RT.augmentation(Routing1.class);
- public final static InstanceIdentifier<Srv6> RT_SRV6 = RT_RT1_AUG.child(Srv6.class);
- public final static InstanceIdentifier<Locators> RT_SRV6_LOCATORS = RT_SRV6.child(Locators.class);
- public final static InstanceIdentifier<Locator> RT_SRV6_LOCS_LOCATOR = RT_SRV6_LOCATORS.child(Locator.class);
- public final static InstanceIdentifier<Locator> LOCATOR = InstanceIdentifier.create(Locator.class);
- public final static InstanceIdentifier<VppSrv6FibLocatorAugment> LOC_FT_AUG =
+ public static final InstanceIdentifier<Routing> RT = InstanceIdentifier.create(Routing.class);
+ public static final InstanceIdentifier<Routing1> RT_RT1_AUG = RT.augmentation(Routing1.class);
+ public static final InstanceIdentifier<Srv6> RT_SRV6 = RT_RT1_AUG.child(Srv6.class);
+ public static final InstanceIdentifier<Locators> RT_SRV6_LOCATORS = RT_SRV6.child(Locators.class);
+ public static final InstanceIdentifier<Locator> RT_SRV6_LOCS_LOCATOR = RT_SRV6_LOCATORS.child(Locator.class);
+ public static final InstanceIdentifier<Locator> LOCATOR = InstanceIdentifier.create(Locator.class);
+ public static final InstanceIdentifier<VppSrv6FibLocatorAugment> LOC_FT_AUG =
LOCATOR.augmentation(VppSrv6FibLocatorAugment.class);
- public final static InstanceIdentifier<FibTable> LOC_FT = LOC_FT_AUG.child(FibTable.class);
+ public static final InstanceIdentifier<FibTable> LOC_FT = LOC_FT_AUG.child(FibTable.class);
- public final static InstanceIdentifier<Prefix> LOC_PREFIX = LOCATOR.child(Prefix.class);
- public final static InstanceIdentifier<Locator1> RT_SRV6_LOCS_LOC_AUG =
+ public static final InstanceIdentifier<Prefix> LOC_PREFIX = LOCATOR.child(Prefix.class);
+ public static final InstanceIdentifier<Locator1> RT_SRV6_LOCS_LOC_AUG =
RT_SRV6_LOCS_LOCATOR.augmentation(Locator1.class);
- public final static InstanceIdentifier<Static> RT_SRV6_LOCS_LOC_STATIC = RT_SRV6_LOCS_LOC_AUG.child(Static.class);
- public final static InstanceIdentifier<LocalSids> RT_SRV6_LOCS_LOC_ST_LOCALSIDS =
+ public static final InstanceIdentifier<Static> RT_SRV6_LOCS_LOC_STATIC = RT_SRV6_LOCS_LOC_AUG.child(Static.class);
+ public static final InstanceIdentifier<LocalSids> RT_SRV6_LOCS_LOC_ST_LOCALSIDS =
RT_SRV6_LOCS_LOC_STATIC.child(LocalSids.class);
- public final static InstanceIdentifier<Sid> RT_SRV6_LOCS_LOC_ST_LS_SID =
+ public static final InstanceIdentifier<Sid> RT_SRV6_LOCS_LOC_ST_LS_SID =
RT_SRV6_LOCS_LOC_ST_LOCALSIDS.child(Sid.class);
- public final static InstanceIdentifier<Encapsulation> RT_SRV6_ENCAP = RT_SRV6.child(Encapsulation.class);
+ public static final InstanceIdentifier<Encapsulation> RT_SRV6_ENCAP = RT_SRV6.child(Encapsulation.class);
- public final static InstanceIdentifier<Sid> SID = InstanceIdentifier.create(Sid.class);
- public final static InstanceIdentifier<End> SID_END = SID.child(End.class);
- public final static InstanceIdentifier<EndX> SID_END_X = SID.child(EndX.class);
- public final static InstanceIdentifier<Paths> SID_END_X_PATHS = SID_END_X.child(Paths.class);
- public final static InstanceIdentifier<Path> SID_END_X_PATHS_PATH = SID_END_X_PATHS.child(Path.class);
- public final static InstanceIdentifier<EndT> SID_END_T = SID.child(EndT.class);
- public final static InstanceIdentifier<EndB6> SID_END_B6 = SID.child(EndB6.class);
- public final static InstanceIdentifier<EndB6Encaps> SID_END_B6ENCAP = SID.child(EndB6Encaps.class);
- public final static InstanceIdentifier<EndBm> SID_END_BM = SID.child(EndBm.class);
- public final static InstanceIdentifier<EndDt4> SID_END_DT4 = SID.child(EndDt4.class);
- public final static InstanceIdentifier<EndDt6> SID_END_DT6 = SID.child(EndDt6.class);
- public final static InstanceIdentifier<EndDt46> SID_END_DT46 = SID.child(EndDt46.class);
- public final static InstanceIdentifier<EndDx2> SID_END_DX2 = SID.child(EndDx2.class);
- public final static InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.srv6.sid.config.end.dx2.Paths>
+ public static final InstanceIdentifier<Sid> SID = InstanceIdentifier.create(Sid.class);
+ public static final InstanceIdentifier<End> SID_END = SID.child(End.class);
+ public static final InstanceIdentifier<EndX> SID_END_X = SID.child(EndX.class);
+ public static final InstanceIdentifier<Paths> SID_END_X_PATHS = SID_END_X.child(Paths.class);
+ public static final InstanceIdentifier<Path> SID_END_X_PATHS_PATH = SID_END_X_PATHS.child(Path.class);
+ public static final InstanceIdentifier<EndT> SID_END_T = SID.child(EndT.class);
+ public static final InstanceIdentifier<EndB6> SID_END_B6 = SID.child(EndB6.class);
+ public static final InstanceIdentifier<EndB6Encaps> SID_END_B6ENCAP = SID.child(EndB6Encaps.class);
+ public static final InstanceIdentifier<EndBm> SID_END_BM = SID.child(EndBm.class);
+ public static final InstanceIdentifier<EndDt4> SID_END_DT4 = SID.child(EndDt4.class);
+ public static final InstanceIdentifier<EndDt6> SID_END_DT6 = SID.child(EndDt6.class);
+ public static final InstanceIdentifier<EndDt46> SID_END_DT46 = SID.child(EndDt46.class);
+ public static final InstanceIdentifier<EndDx2> SID_END_DX2 = SID.child(EndDx2.class);
+ public static final InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.srv6.sid.config.end.dx2.Paths>
SID_END_DX2_PATHS = SID_END_DX2.child(
org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.srv6.sid.config.end.dx2.Paths.class);
- public final static InstanceIdentifier<EndDx4> SID_END_DX4 = SID.child(EndDx4.class);
- public final static InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.multi.paths.v4.Paths>
+ public static final InstanceIdentifier<EndDx4> SID_END_DX4 = SID.child(EndDx4.class);
+ public static final InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.multi.paths.v4.Paths>
SID_END_DX4_PATHS = SID_END_DX4.child(
org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.multi.paths.v4.Paths.class);
- public final static InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.multi.paths.v4.paths.Path>
+ public static final InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.multi.paths.v4.paths.Path>
SID_END_DX4_PATHS_PATH = SID_END_DX4_PATHS.child(
org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.srv6._static.rev180301.multi.paths.v4.paths.Path.class);
- public final static InstanceIdentifier<EndDx6> SID_END_DX6 = SID.child(EndDx6.class);
- public final static InstanceIdentifier<Paths> SID_END_DX6_PATHS = SID_END_DX6.child(Paths.class);
- public final static InstanceIdentifier<Path> SID_END_DX6_PATHS_PATH = SID_END_DX6_PATHS.child(Path.class);
+ public static final InstanceIdentifier<EndDx6> SID_END_DX6 = SID.child(EndDx6.class);
+ public static final InstanceIdentifier<Paths> SID_END_DX6_PATHS = SID_END_DX6.child(Paths.class);
+ public static final InstanceIdentifier<Path> SID_END_DX6_PATHS_PATH = SID_END_DX6_PATHS.child(Path.class);
}
diff --git a/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/read/acl/AclReader.java b/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/read/acl/AclReader.java
index ba3775f9a..75c370c01 100644
--- a/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/read/acl/AclReader.java
+++ b/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/read/acl/AclReader.java
@@ -28,7 +28,6 @@ import org.opendaylight.yang.gen.v1.http.fd.io.hc2vpp.yang.vpp.classifier.acl.re
interface AclReader {
- @Nonnull
default L2Acl readL2Acl(final int l2TableId, @Nonnull final VppClassifierContextManager classifyTableContext,
@Nonnull final MappingContext mappingContext) {
if (l2TableId == ~0) {
@@ -38,7 +37,6 @@ interface AclReader {
.setClassifyTable(classifyTableContext.getTableName(l2TableId, mappingContext)).build();
}
- @Nonnull
default Ip4Acl readIp4Acl(final int ip4TableId, @Nonnull final VppClassifierContextManager classifyTableContext,
@Nonnull final MappingContext mappingContext) {
if (ip4TableId == ~0) {
@@ -48,7 +46,6 @@ interface AclReader {
.setClassifyTable(classifyTableContext.getTableName(ip4TableId, mappingContext)).build();
}
- @Nonnull
default Ip6Acl readIp6Acl(final int ip6TableId, @Nonnull final VppClassifierContextManager classifyTableContext,
@Nonnull final MappingContext mappingContext) {
if (ip6TableId == ~0) {