> Do you need to check it twice? > > Is the second test now redundant?
I agree, the second if statement can be simplified to only check the scope id. > > + > > + if (!(ipv6_addr_type(&sin6->sin6_addr) & IPV6_ADDR_LINKLOCAL)) > > + return 0; > > if ((ipv6_addr_type(&sin6->sin6_addr) & IPV6_ADDR_LINKLOCAL) && > > !sin6->sin6_scope_id) > > return -EINVAL; -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
