[Devel] Re: [PATCH 3/3] sysctl: Error on bad sysctl tables

Eric W. Biederman ebiederm at xmission.com
Thu Aug 9 19:18:53 PDT 2007


YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji at linux-ipv6.org> writes:

> Hello.
>
> In article <m1hcn8a2rq.fsf_-_ at ebiederm.dsl.xmission.com> (at Thu, 09 Aug 2007
> 14:09:29 -0600), ebiederm at xmission.com (Eric W. Biederman) says:
>
>> After going through the kernels sysctl tables several times it has
>> become clear that code review and testing is just not effective in
>> prevent problematic sysctl tables from being used in the stable
>> kernel.  I certainly can't seem to fix the problems as fast as
>> they are introduced.
> :
>> The biggest part of the code is the table of valid binary sysctl
>> entries, but since we have frozen our set of binary sysctls this table
>> should not need to change, and it makes it much easier to detect
>> when someone unintentionally adds a new binary sysctl value.
>
> I don't think everyone needs to have this code, so
> it is better to make it configurable via
> CONFIG_SYSCTL_DEBUG or something..., ...no?

I guess the other thing is.  Except for code size it doesn't matter.
As register_sysctl_table gets called very rarely.

Eric




More information about the Devel mailing list