mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 02:30:34 +02:00 
			
		
		
		
	sctp: add check rto_min and rto_max in sysctl
rto_min should be smaller than rto_max while rto_max should be larger than rto_min. Add two proc_handler for the checking. Suggested-by: Vlad Yasevich <vyasevich@gmail.com> Signed-off-by: Wang Weidong <wangweidong1@huawei.com> Acked-by: Vlad Yasevich <vyasevich@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									85f935d41a
								
							
						
					
					
						commit
						4f3fdf3bc5
					
				
					 1 changed files with 65 additions and 4 deletions
				
			
		| 
						 | 
					@ -61,6 +61,13 @@ static int proc_sctp_do_hmac_alg(struct ctl_table *ctl,
 | 
				
			||||||
				void __user *buffer, size_t *lenp,
 | 
									void __user *buffer, size_t *lenp,
 | 
				
			||||||
 | 
					
 | 
				
			||||||
				loff_t *ppos);
 | 
									loff_t *ppos);
 | 
				
			||||||
 | 
					static int proc_sctp_do_rto_min(struct ctl_table *ctl, int write,
 | 
				
			||||||
 | 
									void __user *buffer, size_t *lenp,
 | 
				
			||||||
 | 
									loff_t *ppos);
 | 
				
			||||||
 | 
					static int proc_sctp_do_rto_max(struct ctl_table *ctl, int write,
 | 
				
			||||||
 | 
									void __user *buffer, size_t *lenp,
 | 
				
			||||||
 | 
									loff_t *ppos);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
static struct ctl_table sctp_table[] = {
 | 
					static struct ctl_table sctp_table[] = {
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
		.procname	= "sctp_mem",
 | 
							.procname	= "sctp_mem",
 | 
				
			||||||
| 
						 | 
					@ -102,17 +109,17 @@ static struct ctl_table sctp_net_table[] = {
 | 
				
			||||||
		.data		= &init_net.sctp.rto_min,
 | 
							.data		= &init_net.sctp.rto_min,
 | 
				
			||||||
		.maxlen		= sizeof(unsigned int),
 | 
							.maxlen		= sizeof(unsigned int),
 | 
				
			||||||
		.mode		= 0644,
 | 
							.mode		= 0644,
 | 
				
			||||||
		.proc_handler	= proc_dointvec_minmax,
 | 
							.proc_handler	= proc_sctp_do_rto_min,
 | 
				
			||||||
		.extra1         = &one,
 | 
							.extra1         = &one,
 | 
				
			||||||
		.extra2         = &timer_max
 | 
							.extra2         = &init_net.sctp.rto_max
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
		.procname	= "rto_max",
 | 
							.procname	= "rto_max",
 | 
				
			||||||
		.data		= &init_net.sctp.rto_max,
 | 
							.data		= &init_net.sctp.rto_max,
 | 
				
			||||||
		.maxlen		= sizeof(unsigned int),
 | 
							.maxlen		= sizeof(unsigned int),
 | 
				
			||||||
		.mode		= 0644,
 | 
							.mode		= 0644,
 | 
				
			||||||
		.proc_handler	= proc_dointvec_minmax,
 | 
							.proc_handler	= proc_sctp_do_rto_max,
 | 
				
			||||||
		.extra1         = &one,
 | 
							.extra1         = &init_net.sctp.rto_min,
 | 
				
			||||||
		.extra2         = &timer_max
 | 
							.extra2         = &timer_max
 | 
				
			||||||
	},
 | 
						},
 | 
				
			||||||
	{
 | 
						{
 | 
				
			||||||
| 
						 | 
					@ -342,6 +349,60 @@ static int proc_sctp_do_hmac_alg(struct ctl_table *ctl,
 | 
				
			||||||
	return ret;
 | 
						return ret;
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static int proc_sctp_do_rto_min(struct ctl_table *ctl, int write,
 | 
				
			||||||
 | 
									void __user *buffer, size_t *lenp,
 | 
				
			||||||
 | 
									loff_t *ppos)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						struct net *net = current->nsproxy->net_ns;
 | 
				
			||||||
 | 
						int new_value;
 | 
				
			||||||
 | 
						struct ctl_table tbl;
 | 
				
			||||||
 | 
						unsigned int min = *(unsigned int *) ctl->extra1;
 | 
				
			||||||
 | 
						unsigned int max = *(unsigned int *) ctl->extra2;
 | 
				
			||||||
 | 
						int ret;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						memset(&tbl, 0, sizeof(struct ctl_table));
 | 
				
			||||||
 | 
						tbl.maxlen = sizeof(unsigned int);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (write)
 | 
				
			||||||
 | 
							tbl.data = &new_value;
 | 
				
			||||||
 | 
						else
 | 
				
			||||||
 | 
							tbl.data = &net->sctp.rto_min;
 | 
				
			||||||
 | 
						ret = proc_dointvec(&tbl, write, buffer, lenp, ppos);
 | 
				
			||||||
 | 
						if (write) {
 | 
				
			||||||
 | 
							if (ret || new_value > max || new_value < min)
 | 
				
			||||||
 | 
								return -EINVAL;
 | 
				
			||||||
 | 
							net->sctp.rto_min = new_value;
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
						return ret;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static int proc_sctp_do_rto_max(struct ctl_table *ctl, int write,
 | 
				
			||||||
 | 
									void __user *buffer, size_t *lenp,
 | 
				
			||||||
 | 
									loff_t *ppos)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
						struct net *net = current->nsproxy->net_ns;
 | 
				
			||||||
 | 
						int new_value;
 | 
				
			||||||
 | 
						struct ctl_table tbl;
 | 
				
			||||||
 | 
						unsigned int min = *(unsigned int *) ctl->extra1;
 | 
				
			||||||
 | 
						unsigned int max = *(unsigned int *) ctl->extra2;
 | 
				
			||||||
 | 
						int ret;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						memset(&tbl, 0, sizeof(struct ctl_table));
 | 
				
			||||||
 | 
						tbl.maxlen = sizeof(unsigned int);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if (write)
 | 
				
			||||||
 | 
							tbl.data = &new_value;
 | 
				
			||||||
 | 
						else
 | 
				
			||||||
 | 
							tbl.data = &net->sctp.rto_max;
 | 
				
			||||||
 | 
						ret = proc_dointvec(&tbl, write, buffer, lenp, ppos);
 | 
				
			||||||
 | 
						if (write) {
 | 
				
			||||||
 | 
							if (ret || new_value > max || new_value < min)
 | 
				
			||||||
 | 
								return -EINVAL;
 | 
				
			||||||
 | 
							net->sctp.rto_max = new_value;
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
						return ret;
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
int sctp_sysctl_net_register(struct net *net)
 | 
					int sctp_sysctl_net_register(struct net *net)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	struct ctl_table *table;
 | 
						struct ctl_table *table;
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue