[
https://issues.apache.org/jira/browse/CLOUDSTACK-8749?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14703284#comment-14703284
]
ASF GitHub Bot commented on CLOUDSTACK-8749:
--------------------------------------------
Github user jburwell commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/718#discussion_r37435429
--- Diff:
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java
---
@@ -287,7 +287,7 @@ private void deleteExitingLinkLocalRouteTable(String
linkLocalBr) {
for (String line : lines) {
String[] tokens = line.split(" ");
if (!tokens[2].equalsIgnoreCase(linkLocalBr)) {
--- End diff --
Consider the following as a bit of extra safety in the event of malformed
input beginning at line 289:
```
if (tokens != null && tokens.length < 2) {
continue;
}
final String device = tokens[2];
if (!Strings.isNullOrEmpty(device) &&
!device.equalsIgnoreCase(linkLocalBr)) {
```
> KVM: link local route cleanup
> -----------------------------
>
> Key: CLOUDSTACK-8749
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8749
> Project: CloudStack
> Issue Type: Bug
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Reporter: Rohit Yadav
> Assignee: Rohit Yadav
> Fix For: 4.6.0, 4.5.3
>
>
> The KVM agent attempts to delete link local interfaces whenever found, so
> they don't conflict with the system vm's link local route on cloud0, however
> it doesn't specify which device to delete the route for.
> The aim is to fix the edge case which (unlikely) can try to delete existing
> cloud0 routes/interface.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
