While experimenting with a CodeSonar plugin we develop, we noticed a
potential inconsistency in file "kernel/sysctl.c" line 1339, initialization of "vm_table" array.

{
    .procname	= "dirtytime_expire_seconds",
    .data	= &dirtytime_expire_interval,
    .maxlen	= sizeof(dirty_expire_interval), //HERE
    .mode	= 0644,
    .proc_handler = dirtytime_interval_handler,
    .extra1	= &zero,
},

Shoudn't "dirtytime_expire_interval" be used instead of "dirty_expire_interval"? Yes, it looks harmless because the variables have the same type. However, the real intention looks to be the usage of sizeof(dirtytime_expire_interval).

Thanks,
Petru Mihancea