fix(lb): removing extra frontends and backends resulted in segfault (#153)

nox-404 · web-flow · commit 1643637b0e13 · 2024-01-12T17:12:42.000+01:00
diff --git a/scaleway/loadbalancers.go b/scaleway/loadbalancers.go
@@ -1958,7 +1958,7 @@ func compareFrontends(got []*scwlb.Frontend, want map[int32]*scwlb.Frontend) fro
 				keep[target.InboundPort] = current
 			}
 		} else {
-			remove[target.InboundPort] = current
+			remove[current.InboundPort] = current
 		}
 	}
 
@@ -2007,7 +2007,7 @@ func compareBackends(got []*scwlb.Backend, want map[int32]*scwlb.Backend) backen
 				keep[target.ForwardPort] = current
 			}
 		} else {
-			remove[target.ForwardPort] = current
+			remove[current.ForwardPort] = current
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -1958,7 +1958,7 @@ func compareFrontends(got []scwlb.Frontend, want map[int32]scwlb.Frontend) fro`
`1958`	`1958`	`keep[target.InboundPort] = current`
`1959`	`1959`	`}`
`1960`	`1960`	`} else {`
`1961`		`- remove[target.InboundPort] = current`
	`1961`	`+ remove[current.InboundPort] = current`
`1962`	`1962`	`}`
`1963`	`1963`	`}`
`1964`	`1964`
`@@ -2007,7 +2007,7 @@ func compareBackends(got []scwlb.Backend, want map[int32]scwlb.Backend) backen`
`2007`	`2007`	`keep[target.ForwardPort] = current`
`2008`	`2008`	`}`
`2009`	`2009`	`} else {`
`2010`		`- remove[target.ForwardPort] = current`
	`2010`	`+ remove[current.ForwardPort] = current`
`2011`	`2011`	`}`
`2012`	`2012`	`}`
`2013`	`2013`