From cd291463179c03c26fd614f2a6dfdbe75f58b92e Mon Sep 17 00:00:00 2001 From: FengyunPan Date: Wed, 12 Jul 2017 17:28:58 +0800 Subject: [PATCH] Fix panic of DeleteRoute() Fix #48800 It should be 'addr_pairs', not 'routes'. --- pkg/cloudprovider/providers/openstack/openstack_routes.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/cloudprovider/providers/openstack/openstack_routes.go b/pkg/cloudprovider/providers/openstack/openstack_routes.go index 28e5160a599..2cb0cd95ff0 100644 --- a/pkg/cloudprovider/providers/openstack/openstack_routes.go +++ b/pkg/cloudprovider/providers/openstack/openstack_routes.go @@ -262,8 +262,8 @@ func (r *Routes) DeleteRoute(clusterName string, route *cloudprovider.Route) err if index != -1 { // Delete element `index` - addr_pairs[index] = addr_pairs[len(routes)-1] - addr_pairs = addr_pairs[:len(routes)-1] + addr_pairs[index] = addr_pairs[len(addr_pairs)-1] + addr_pairs = addr_pairs[:len(addr_pairs)-1] unwind, err := updateAllowedAddressPairs(r.network, &port, addr_pairs) if err != nil {