diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2014-08-27 06:25:23 +0930 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2014-08-27 21:54:10 +0930 |
commit | 7a486d3781295b5298cbf9556928a76d26896863 (patch) | |
tree | 7a4d6592ed62a04c125d654f5ea6c15669c67b36 /include/linux/moduleparam.h | |
parent | fc9740cebc3ab7c65f3c5f6ce0caf3e4969013ca (diff) | |
download | linux-stable-7a486d3781295b5298cbf9556928a76d26896863.tar.gz linux-stable-7a486d3781295b5298cbf9556928a76d26896863.tar.bz2 linux-stable-7a486d3781295b5298cbf9556928a76d26896863.zip |
param: check for tainting before calling set op.
This means every set op doesn't need to call it, and it can move into
params.c.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'include/linux/moduleparam.h')
-rw-r--r-- | include/linux/moduleparam.h | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h index 9531f9f9729e..593501996574 100644 --- a/include/linux/moduleparam.h +++ b/include/linux/moduleparam.h @@ -374,22 +374,6 @@ static inline void destroy_params(const struct kernel_param *params, #define __param_check(name, p, type) \ static inline type __always_unused *__check_##name(void) { return(p); } -/** - * param_check_unsafe - Warn and taint the kernel if setting dangerous options. - * - * This gets called from all the standard param setters, but can be used from - * custom setters as well. - */ -static inline void -param_check_unsafe(const struct kernel_param *kp) -{ - if (kp->flags & KERNEL_PARAM_FL_UNSAFE) { - pr_warn("Setting dangerous option %s - tainting kernel\n", - kp->name); - add_taint(TAINT_USER, LOCKDEP_STILL_OK); - } -} - extern struct kernel_param_ops param_ops_byte; extern int param_set_byte(const char *val, const struct kernel_param *kp); extern int param_get_byte(char *buffer, const struct kernel_param *kp); |