[PATCH net-next] net/ipv6: Enforce limits for accept_unsolicited_na sysctl

Arun Ajith S posted 1 patch 4 years ago
net/ipv6/addrconf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH net-next] net/ipv6: Enforce limits for accept_unsolicited_na sysctl
Posted by Arun Ajith S 4 years ago
Fix mistake in the original patch where limits were specified but the
handler didn't take care of the limits.

Signed-off-by: Arun Ajith S <aajith@arista.com>
---
 net/ipv6/addrconf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c
index 6473dc84b71d..f01b8a3e1952 100644
--- a/net/ipv6/addrconf.c
+++ b/net/ipv6/addrconf.c
@@ -7043,7 +7043,7 @@ static const struct ctl_table addrconf_sysctl[] = {
 		.data		= &ipv6_devconf.accept_unsolicited_na,
 		.maxlen		= sizeof(int),
 		.mode		= 0644,
-		.proc_handler	= proc_dointvec,
+		.proc_handler	= proc_dointvec_minmax,
 		.extra1		= (void *)SYSCTL_ZERO,
 		.extra2		= (void *)SYSCTL_ONE,
 	},
-- 
2.27.0
Re: [PATCH net-next] net/ipv6: Enforce limits for accept_unsolicited_na sysctl
Posted by patchwork-bot+netdevbpf@kernel.org 4 years ago
Hello:

This patch was applied to netdev/net-next.git (master)
by David S. Miller <davem@davemloft.net>:

On Tue, 19 Apr 2022 10:59:10 +0000 you wrote:
> Fix mistake in the original patch where limits were specified but the
> handler didn't take care of the limits.
> 
> Signed-off-by: Arun Ajith S <aajith@arista.com>
> ---
>  net/ipv6/addrconf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Here is the summary with links:
  - [net-next] net/ipv6: Enforce limits for accept_unsolicited_na sysctl
    https://git.kernel.org/netdev/net-next/c/d09d3ec03f02

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
Re: [PATCH net-next] net/ipv6: Enforce limits for accept_unsolicited_na sysctl
Posted by David Ahern 4 years ago
On 4/19/22 4:59 AM, Arun Ajith S wrote:
> Fix mistake in the original patch where limits were specified but the
> handler didn't take care of the limits.
> 
> Signed-off-by: Arun Ajith S <aajith@arista.com>
> ---
>  net/ipv6/addrconf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 


Reviewed-by: David Ahern <dsahern@kernel.org>