table: check paths' equivalence deeper

if we add same path multiple times by the following gobgp cli commands,

$ gobgp global rib add 10.0.0.0/24
$ gobgp global rib add 10.0.0.0/24
$ gobgp global rib add 10.0.0.0/24

current implementation sends multiple update messages to peers
even through the content of the path is same.
This is due to the shallow equivalence checking.

This patch checks paths' equivalence deeper by adding Equal() method to Path.

Signed-off-by: ISHIDA Wataru <ishida.wataru@lab.ntt.co.jp>

1 files changed, 1 insertions, 1 deletions

diff --git a/table/destination.go b/table/destination.go
index dc6e073f..704cc865 100644
--- a/table/destination.go
+++ b/table/destination.go
@@ -92,7 +92,7 @@ func (dd *Destination) ToApiStruct() *api.Destination {
 
 	idx := func() int {
 		for i, p := range dd.knownPathList {
-			if p == dd.GetBestPath() {
+			if dd.GetBestPath().Equal(p) {
 				return i
 			}
 		}