soreuseport: fix use of uid in tb->fastuid
authorTom Herbert <therbert@google.com>
Sat, 26 Jan 2013 07:50:54 +0000 (07:50 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 27 Jan 2013 06:01:42 +0000 (01:01 -0500)
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: Tom Herbert <therbert@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/inet_connection_sock.c

index 8bb623d..11cb497 100644 (file)
@@ -204,7 +204,8 @@ tb_found:
                        ret = 1;
                        if (inet_csk(sk)->icsk_af_ops->bind_conflict(sk, tb, true)) {
                                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) {
                                        spin_unlock(&head->lock);
                                        goto again;
@@ -227,19 +228,15 @@ tb_not_found:
                if (sk->sk_reuseport) {
                        tb->fastreuseport = 1;
                        tb->fastuid = uid;
-               } else {
+               } else
                        tb->fastreuseport = 0;
-                       tb->fastuid = 0;
-               }
        } else {
                if (tb->fastreuse &&
                    (!sk->sk_reuse || sk->sk_state == TCP_LISTEN))
                        tb->fastreuse = 0;
                if (tb->fastreuseport &&
-                   (!sk->sk_reuseport || !uid_eq(tb->fastuid, uid))) {
+                   (!sk->sk_reuseport || !uid_eq(tb->fastuid, uid)))
                        tb->fastreuseport = 0;
-                       tb->fastuid = 0;
-               }
        }
 success:
        if (!inet_csk(sk)->icsk_bind_hash)