From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Barmann Subject: [PATCH v2 net-next] sock: Reset dst when changing sk_mark via setsockopt Date: Wed, 7 Nov 2018 22:55:52 -0600 Message-ID: <20181108045552.GA24562@konacove.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii To: netdev@vger.kernel.org Return-path: Received: from konacove.com ([69.16.156.219]:47676 "EHLO konacove.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728841AbeKHO3a (ORCPT ); Thu, 8 Nov 2018 09:29:30 -0500 Content-Disposition: inline Sender: netdev-owner@vger.kernel.org List-ID: When setting the SO_MARK socket option, the dst needs to be reset so that a new route lookup is performed. This fixes the case where an application wants to change routing by setting a new sk_mark. If this is done after some packets have already been sent, the dst is cached and has no effect. Signed-off-by: David Barmann --- net/core/sock.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/net/core/sock.c b/net/core/sock.c index 7b304e454a38..c74b10be86cb 100644 --- a/net/core/sock.c +++ b/net/core/sock.c @@ -952,10 +952,12 @@ int sock_setsockopt(struct socket *sock, int level, int optname, clear_bit(SOCK_PASSSEC, &sock->flags); break; case SO_MARK: - if (!ns_capable(sock_net(sk)->user_ns, CAP_NET_ADMIN)) + if (!ns_capable(sock_net(sk)->user_ns, CAP_NET_ADMIN)) { ret = -EPERM; - else + } else { sk->sk_mark = val; + sk_dst_reset(sk); + } break; case SO_RXQ_OVFL: -- 2.14.5