Author: markj
Date: Sat Nov 17 20:02:57 2018
New Revision: 340512
URL: https://svnweb.freebsd.org/changeset/base/340512

Log:
  MFC r339598:
  Fix style bugs in in6_pcblookup_lbgroup().

Modified:
  stable/12/sys/netinet6/in6_pcb.c
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/sys/netinet6/in6_pcb.c
==============================================================================
--- stable/12/sys/netinet6/in6_pcb.c    Sat Nov 17 20:01:35 2018        
(r340511)
+++ stable/12/sys/netinet6/in6_pcb.c    Sat Nov 17 20:02:57 2018        
(r340512)
@@ -873,10 +873,9 @@ in6_pcblookup_lbgroup(const struct inpcbinfo *pcbinfo,
     const struct in6_addr *laddr, uint16_t lport, const struct in6_addr *faddr,
     uint16_t fport, int lookupflags)
 {
-       struct inpcb *local_wild = NULL;
+       struct inpcb *local_wild;
        const struct inpcblbgrouphead *hdr;
        struct inpcblbgroup *grp;
-       struct inpcblbgroup *grp_local_wild;
        uint32_t idx;
 
        INP_HASH_LOCK_ASSERT(pcbinfo);
@@ -893,33 +892,24 @@ in6_pcblookup_lbgroup(const struct inpcbinfo *pcbinfo,
         * - Load balanced group does not contain jailed sockets.
         * - Load balanced does not contain IPv4 mapped INET6 wild sockets.
         */
+       local_wild = NULL;
        CK_LIST_FOREACH(grp, hdr, il_list) {
 #ifdef INET
                if (!(grp->il_vflag & INP_IPV6))
                        continue;
 #endif
-               if (grp->il_lport == lport) {
-                       idx = 0;
-                       int pkt_hash = INP_PCBLBGROUP_PKTHASH(
-                           INP6_PCBHASHKEY(faddr), lport, fport);
+               if (grp->il_lport != lport)
+                       continue;
 
-                       idx = pkt_hash % grp->il_inpcnt;
-
-                       if (IN6_ARE_ADDR_EQUAL(&grp->il6_laddr, laddr)) {
-                               return (grp->il_inp[idx]);
-                       } else {
-                               if (IN6_IS_ADDR_UNSPECIFIED(&grp->il6_laddr) &&
-                                   (lookupflags & INPLOOKUP_WILDCARD)) {
-                                       local_wild = grp->il_inp[idx];
-                                       grp_local_wild = grp;
-                               }
-                       }
-               }
+               idx = INP_PCBLBGROUP_PKTHASH(INP6_PCBHASHKEY(faddr), lport,
+                   fport) % grp->il_inpcnt;
+               if (IN6_ARE_ADDR_EQUAL(&grp->il6_laddr, laddr))
+                       return (grp->il_inp[idx]);
+               if (IN6_IS_ADDR_UNSPECIFIED(&grp->il6_laddr) &&
+                   (lookupflags & INPLOOKUP_WILDCARD) != 0)
+                       local_wild = grp->il_inp[idx];
        }
-       if (local_wild != NULL) {
-               return (local_wild);
-       }
-       return (NULL);
+       return (local_wild);
 }
 
 #ifdef PCBGROUP
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to