netfilter: rpfilter: Convert rpfilter_lookup_reverse to new dev helper
Convert rpfilter_lookup_reverse to the new device checking helper and remove the duplicate code. Signed-off-by: David Ahern <dsahern@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									78f2756c5f
								
							
						
					
					
						commit
						91a178258a
					
				| @ -36,7 +36,6 @@ static bool rpfilter_lookup_reverse(struct net *net, struct flowi4 *fl4, | ||||
| 				const struct net_device *dev, u8 flags) | ||||
| { | ||||
| 	struct fib_result res; | ||||
| 	bool dev_match; | ||||
| 	int ret __maybe_unused; | ||||
| 
 | ||||
| 	if (fib_lookup(net, fl4, &res, FIB_LOOKUP_IGNORE_LINKSTATE)) | ||||
| @ -46,21 +45,7 @@ static bool rpfilter_lookup_reverse(struct net *net, struct flowi4 *fl4, | ||||
| 		if (res.type != RTN_LOCAL || !(flags & XT_RPFILTER_ACCEPT_LOCAL)) | ||||
| 			return false; | ||||
| 	} | ||||
| 	dev_match = false; | ||||
| #ifdef CONFIG_IP_ROUTE_MULTIPATH | ||||
| 	for (ret = 0; ret < res.fi->fib_nhs; ret++) { | ||||
| 		struct fib_nh *nh = &res.fi->fib_nh[ret]; | ||||
| 
 | ||||
| 		if (nh->nh_dev == dev) { | ||||
| 			dev_match = true; | ||||
| 			break; | ||||
| 		} | ||||
| 	} | ||||
| #else | ||||
| 	if (FIB_RES_DEV(res) == dev) | ||||
| 		dev_match = true; | ||||
| #endif | ||||
| 	return dev_match || flags & XT_RPFILTER_LOOSE; | ||||
| 	return fib_info_nh_uses_dev(res.fi, dev) || flags & XT_RPFILTER_LOOSE; | ||||
| } | ||||
| 
 | ||||
| static bool | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user