Skip to content

Commit

Permalink
ignore endpoints with >1 adjacency in process rendering
Browse files Browse the repository at this point in the history
This eliminates the worst effects of #2665.
  • Loading branch information
rade committed Jun 29, 2017
1 parent 247c365 commit 708931f
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions render/process.go
Original file line number Diff line number Diff line change
@@ -123,6 +123,13 @@ func MapEndpoint2Process(n report.Node, local report.Networks) report.Nodes {
return report.Nodes{}
}

if len(n.Adjacency) > 1 {
// We cannot be sure that the pid is associated with all the
// connections. It is better to drop such an endpoint than
// risk rendering bogus connections.
return report.Nodes{}
}

id := report.MakeProcessNodeID(report.ExtractHostID(n), pid)
node := NewDerivedNode(id, n).WithTopology(report.Process)
node.Latest = node.Latest.Set(process.PID, timestamp, pid)

0 comments on commit 708931f

Please sign in to comment.