Skip to content

Commit ab98a1f

Browse files
committed
Fix: resolve a NPE when removing VLAN typed openstack network
Change-Id: I55c4f215211a2ecc5353ea9033dca41cbbd81edf
1 parent 40f032a commit ab98a1f

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

apps/openstacknetworking/app/src/main/java/org/onosproject/openstacknetworking/impl/OpenstackSwitchingArpHandler.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -481,7 +481,7 @@ private void setRemoteArpRequestRuleForTunnel(InstancePort port, boolean install
481481
.matchEthType(EthType.EtherType.ARP.ethType().toShort())
482482
.matchArpOp(ARP.OP_REQUEST)
483483
.matchArpTpa(port.ipAddress().getIp4Address())
484-
.matchTunnelId(Long.valueOf(segId))
484+
.matchTunnelId(Long.parseLong(segId))
485485
.build();
486486

487487
setRemoteArpTreatmentForTunnel(selector, port, localNode, install);
@@ -954,15 +954,15 @@ private void setVnetArpRule(Network network, boolean install) {
954954

955955
if (netType != NetworkType.LOCAL && netType != NetworkType.FLAT
956956
&& netType != NetworkType.VLAN) {
957-
String segId = osNetworkService.segmentId(netId);
957+
String segId = network.getProviderSegID();
958958
osNodeService.completeNodes(COMPUTE)
959959
.forEach(node -> {
960960
setBaseVnetArpRuleForBroadcastMode(node, segId,
961961
netId, true, install);
962962
});
963963
}
964964
if (netType == NetworkType.VLAN) {
965-
String segId = osNetworkService.segmentId(netId);
965+
String segId = network.getProviderSegID();
966966
osNodeService.completeNodes(COMPUTE)
967967
.forEach(node -> {
968968
setBaseVnetArpRuleForBroadcastMode(

0 commit comments

Comments
 (0)