Re: [RFC] kernel/sysctl.c: return -EINVAL when write invalid val to ulong type sysctl

From	Yisheng Xie <xieyisheng1@huawei.com>
Newsgroups	linux.kernel
Subject	Re: [RFC] kernel/sysctl.c: return -EINVAL when write invalid val to ulong type sysctl
Date	2016-11-29 11:30 +0100
Message-ID	<sINPI-7eO-17@gated-at.bofh.it> (permalink)
References	<sHHjk-4Am-13@gated-at.bofh.it> <sHUqe-4EY-15@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw


On 2016/11/27 7:15, subashab@codeaurora.org wrote:
> On 2016-11-26 02:13, Yisheng Xie wrote:
>> ----------------------
>> diff --git a/kernel/sysctl.c b/kernel/sysctl.c
>> index 706309f..40e9285 100644
>> --- a/kernel/sysctl.c
>> +++ b/kernel/sysctl.c
>> @@ -2485,10 +2485,14 @@ static int __do_proc_doulongvec_minmax(void
>> *data, struct ctl_table *table, int
>>                                              sizeof(proc_wspace_sep), NULL);
>>                         if (err)
>>                                 break;
>> -                       if (neg)
>> -                               continue;
>> -                       if ((min && val < *min) || (max && val > *max))
>> -                               continue;
>> +                       if (neg) {
>> +                               err = -EINVAL;
>> +                               break;
>> +                       }
>> +                       if ((min && val < *min) || (max && val > *max)) {
>> +                               err = -EINVAL;
>> +                               break;
>> +                       }
>>                         *i = val;
>>                 } else {
>>                         val = convdiv * (*i) / convmul;
> 
> Agree, this should be similar to proc_douintvec
> 
> root@vm:~# echo 8192 > /proc/sys/net/core/xfrm_aevent_rseqth
> root@vm:~# cat /proc/sys/net/core/xfrm_aevent_rseqth
> 8192
> root@vm:~# echo -1 > /proc/sys/net/core/xfrm_aevent_rseqth
> -bash: echo: write error: Invalid argument
> root@vm:~# cat /proc/sys/net/core/xfrm_aevent_rseqth
> 8192
> 
Hi,
Thank you for your reply,
then I will write a formal patch to see whether it is acceptable.

Thanks,
Yisheng Xie
> -- 
> Qualcomm Innovation Center, Inc.
> The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
> 
> .
>

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[RFC] kernel/sysctl.c: return -EINVAL when write invalid val to ulong  type sysctl Yisheng Xie <xieyisheng1@huawei.com> - 2016-11-26 10:20 +0100
  Re: [RFC] kernel/sysctl.c: return -EINVAL when write invalid val to  ulong type sysctl subashab@codeaurora.org - 2016-11-27 00:20 +0100
    Re: [RFC] kernel/sysctl.c: return -EINVAL when write invalid val to  ulong type sysctl Yisheng Xie <xieyisheng1@huawei.com> - 2016-11-29 11:30 +0100

csiph-web