udp: Neaten function pointer calls and add braces
authorJoe Perches <joe@perches.com>
Wed, 12 Nov 2014 05:59:20 +0000 (21:59 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 12 Nov 2014 19:51:59 +0000 (14:51 -0500)
Standardize function pointer uses.

Convert calling style from:
(*foo)(args...);
to:
foo(args...);

Other miscellanea:

o Add braces around loops with single ifs on multiple lines
o Realign arguments around these functions
o Invert logic in if to return immediately.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/udp.c

index 1b6e9d5fadef11d1aa10023dae5cf24d9a365b40..4a16b9129079791e160634f4dfdcd0302beab568 100644 (file)
@@ -144,7 +144,7 @@ static int udp_lib_lport_inuse(struct net *net, __u16 num,
        struct hlist_nulls_node *node;
        kuid_t uid = sock_i_uid(sk);
 
-       sk_nulls_for_each(sk2, node, &hslot->head)
+       sk_nulls_for_each(sk2, node, &hslot->head) {
                if (net_eq(sock_net(sk2), net) &&
                    sk2 != sk &&
                    (bitmap || udp_sk(sk2)->udp_port_hash == num) &&
@@ -152,14 +152,13 @@ static int udp_lib_lport_inuse(struct net *net, __u16 num,
                    (!sk2->sk_bound_dev_if || !sk->sk_bound_dev_if ||
                     sk2->sk_bound_dev_if == sk->sk_bound_dev_if) &&
                    (!sk2->sk_reuseport || !sk->sk_reuseport ||
-                     !uid_eq(uid, sock_i_uid(sk2))) &&
-                   (*saddr_comp)(sk, sk2)) {
-                       if (bitmap)
-                               __set_bit(udp_sk(sk2)->udp_port_hash >> log,
-                                         bitmap);
-                       else
+                    !uid_eq(uid, sock_i_uid(sk2))) &&
+                   saddr_comp(sk, sk2)) {
+                       if (!bitmap)
                                return 1;
+                       __set_bit(udp_sk(sk2)->udp_port_hash >> log, bitmap);
                }
+       }
        return 0;
 }
 
@@ -168,10 +167,10 @@ static int udp_lib_lport_inuse(struct net *net, __u16 num,
  * can insert/delete a socket with local_port == num
  */
 static int udp_lib_lport_inuse2(struct net *net, __u16 num,
-                              struct udp_hslot *hslot2,
-                              struct sock *sk,
-                              int (*saddr_comp)(const struct sock *sk1,
-                                                const struct sock *sk2))
+                               struct udp_hslot *hslot2,
+                               struct sock *sk,
+                               int (*saddr_comp)(const struct sock *sk1,
+                                                 const struct sock *sk2))
 {
        struct sock *sk2;
        struct hlist_nulls_node *node;
@@ -179,7 +178,7 @@ static int udp_lib_lport_inuse2(struct net *net, __u16 num,
        int res = 0;
 
        spin_lock(&hslot2->lock);
-       udp_portaddr_for_each_entry(sk2, node, &hslot2->head)
+       udp_portaddr_for_each_entry(sk2, node, &hslot2->head) {
                if (net_eq(sock_net(sk2), net) &&
                    sk2 != sk &&
                    (udp_sk(sk2)->udp_port_hash == num) &&
@@ -187,11 +186,12 @@ static int udp_lib_lport_inuse2(struct net *net, __u16 num,
                    (!sk2->sk_bound_dev_if || !sk->sk_bound_dev_if ||
                     sk2->sk_bound_dev_if == sk->sk_bound_dev_if) &&
                    (!sk2->sk_reuseport || !sk->sk_reuseport ||
-                     !uid_eq(uid, sock_i_uid(sk2))) &&
-                   (*saddr_comp)(sk, sk2)) {
+                    !uid_eq(uid, sock_i_uid(sk2))) &&
+                   saddr_comp(sk, sk2)) {
                        res = 1;
                        break;
                }
+       }
        spin_unlock(&hslot2->lock);
        return res;
 }
@@ -206,8 +206,8 @@ static int udp_lib_lport_inuse2(struct net *net, __u16 num,
  *                   with NULL address
  */
 int udp_lib_get_port(struct sock *sk, unsigned short snum,
-                      int (*saddr_comp)(const struct sock *sk1,
-                                        const struct sock *sk2),
+                    int (*saddr_comp)(const struct sock *sk1,
+                                      const struct sock *sk2),
                     unsigned int hash2_nulladdr)
 {
        struct udp_hslot *hslot, *hslot2;
@@ -2033,7 +2033,7 @@ int udp_lib_setsockopt(struct sock *sk, int level, int optname,
                } else {
                        up->corkflag = 0;
                        lock_sock(sk);
-                       (*push_pending_frames)(sk);
+                       push_pending_frames(sk);
                        release_sock(sk);
                }
                break;