Skip to content

K
k3s
Commit 5ed87346 authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub
Browse files
Options

Merge pull request #48801 from FengyunPan/fix-panic

Automatic merge from submit-queue (batch tested with PRs 48594, 47042, 48801, 48641, 48243) Fix panic of DeleteRoute() Fix #48800 It should be 'addr_pairs', not 'routes'. **Release note**: ```release-note NONE ```
parents 0a44e5b5 cd291463
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
pkg/cloudprovider/providers/openstack/openstack_routes.go
View file @ 5ed87346
...@@ -262,8 +262,8 @@ func (r *Routes) DeleteRoute(clusterName string, route *cloudprovider.Route) err ...@@ -262,8 +262,8 @@ func (r *Routes) DeleteRoute(clusterName string, route *cloudprovider.Route) err
if index != -1 { if index != -1 {
// Delete element `index` // Delete element `index`
addr_pairs[index] = addr_pairs[len(routes)-1] addr_pairs[index] = addr_pairs[len(addr_pairs)-1]
addr_pairs = addr_pairs[:len(routes)-1] addr_pairs = addr_pairs[:len(addr_pairs)-1]
unwind, err := updateAllowedAddressPairs(r.network, &port, addr_pairs) unwind, err := updateAllowedAddressPairs(r.network, &port, addr_pairs)
if err != nil { if err != nil {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment