Use hlist_entry_safe() instead of open-coding it.
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Reviewed-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
                struct net_bridge_port *port, *lport, *rport;
 
                lport = p ? p->port : NULL;
-               rport = rp ? hlist_entry(rp, struct net_bridge_port, rlist) :
-                            NULL;
+               rport = hlist_entry_safe(rp, struct net_bridge_port, rlist);
 
                if ((unsigned long)lport > (unsigned long)rport) {
                        port = lport;