Commit c6ef006b authored by Vlad Yasevich's avatar Vlad Yasevich Committed by David S. Miller

sctp: Release all routes when processing acks ADD_IP or DEL_IP

When processing an ACK for ADD_IP parameter, we only release
the routes on non-active transports.  This can cause a wrong
source address to be used.  We can release the routes and
cause new route lookups and source address selection so that
new addresses can be used as source.  Additionally, we don't need
to lookup routes for all transports at the same time.  We can let
the transmit code path update the cached route when the transport
actually sends something.
Signed-off-by: default avatarVlad Yasevich <vladislav.yasevich@hp.com>
Signed-off-by: default avatarWei Yongjun <yjwei@cn.fujitsu.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ee9cbaca
...@@ -3193,11 +3193,8 @@ static void sctp_asconf_param_success(struct sctp_association *asoc, ...@@ -3193,11 +3193,8 @@ static void sctp_asconf_param_success(struct sctp_association *asoc,
local_bh_enable(); local_bh_enable();
list_for_each_entry(transport, &asoc->peer.transport_addr_list, list_for_each_entry(transport, &asoc->peer.transport_addr_list,
transports) { transports) {
if (transport->state == SCTP_ACTIVE)
continue;
dst_release(transport->dst); dst_release(transport->dst);
sctp_transport_route(transport, NULL, transport->dst = NULL;
sctp_sk(asoc->base.sk));
} }
break; break;
case SCTP_PARAM_DEL_IP: case SCTP_PARAM_DEL_IP:
...@@ -3207,8 +3204,7 @@ static void sctp_asconf_param_success(struct sctp_association *asoc, ...@@ -3207,8 +3204,7 @@ static void sctp_asconf_param_success(struct sctp_association *asoc,
list_for_each_entry(transport, &asoc->peer.transport_addr_list, list_for_each_entry(transport, &asoc->peer.transport_addr_list,
transports) { transports) {
dst_release(transport->dst); dst_release(transport->dst);
sctp_transport_route(transport, NULL, transport->dst = NULL;
sctp_sk(asoc->base.sk));
} }
break; break;
default: default:
......
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