Commit 9c5e0c0b authored by Tom Herbert's avatar Tom Herbert Committed by David S. Miller

soreuseport: fix use of uid in tb->fastuid

Fix a reported compilation error where ia variable of type kuid_t
was being set to zero.

Eliminate two instances of setting tb->fastuid to zero.  tb->fastuid is
only used if tb->fastreuseport is set, so there should be no problem if
tb->fastuid is not initialized (when tb->fastreuesport is zero).
Signed-off-by: default avatarTom Herbert <therbert@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5588d374
...@@ -204,7 +204,8 @@ int inet_csk_get_port(struct sock *sk, unsigned short snum) ...@@ -204,7 +204,8 @@ int inet_csk_get_port(struct sock *sk, unsigned short snum)
ret = 1; ret = 1;
if (inet_csk(sk)->icsk_af_ops->bind_conflict(sk, tb, true)) { if (inet_csk(sk)->icsk_af_ops->bind_conflict(sk, tb, true)) {
if (((sk->sk_reuse && sk->sk_state != TCP_LISTEN) || if (((sk->sk_reuse && sk->sk_state != TCP_LISTEN) ||
(sk->sk_reuseport && uid_eq(tb->fastuid, uid))) && (tb->fastreuseport > 0 &&
sk->sk_reuseport && uid_eq(tb->fastuid, uid))) &&
smallest_size != -1 && --attempts >= 0) { smallest_size != -1 && --attempts >= 0) {
spin_unlock(&head->lock); spin_unlock(&head->lock);
goto again; goto again;
...@@ -227,19 +228,15 @@ int inet_csk_get_port(struct sock *sk, unsigned short snum) ...@@ -227,19 +228,15 @@ int inet_csk_get_port(struct sock *sk, unsigned short snum)
if (sk->sk_reuseport) { if (sk->sk_reuseport) {
tb->fastreuseport = 1; tb->fastreuseport = 1;
tb->fastuid = uid; tb->fastuid = uid;
} else { } else
tb->fastreuseport = 0; tb->fastreuseport = 0;
tb->fastuid = 0;
}
} else { } else {
if (tb->fastreuse && if (tb->fastreuse &&
(!sk->sk_reuse || sk->sk_state == TCP_LISTEN)) (!sk->sk_reuse || sk->sk_state == TCP_LISTEN))
tb->fastreuse = 0; tb->fastreuse = 0;
if (tb->fastreuseport && if (tb->fastreuseport &&
(!sk->sk_reuseport || !uid_eq(tb->fastuid, uid))) { (!sk->sk_reuseport || !uid_eq(tb->fastuid, uid)))
tb->fastreuseport = 0; tb->fastreuseport = 0;
tb->fastuid = 0;
}
} }
success: success:
if (!inet_csk(sk)->icsk_bind_hash) if (!inet_csk(sk)->icsk_bind_hash)
......
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