module: Do not paper over type mismatches in module_param_call()
The module_param_call() macro was explicitly casting the .set and .get function prototypes away. This can lead to hard-to-find type mismatches. Now that all the function prototypes have been fixed tree-wide, we can drop these casts, and use named initializers too. Signed-off-by: Kees Cook <keescook@chromium.org> Signed-off-by: Jessica Yu <jeyu@kernel.org>
This commit is contained in:
parent
e4dca7b7aa
commit
ece1996a21
1 changed files with 2 additions and 2 deletions
|
@ -227,9 +227,9 @@ struct kparam_array
|
|||
VERIFY_OCTAL_PERMISSIONS(perm), level, flags, { arg } }
|
||||
|
||||
/* Obsolete - use module_param_cb() */
|
||||
#define module_param_call(name, set, get, arg, perm) \
|
||||
#define module_param_call(name, _set, _get, arg, perm) \
|
||||
static const struct kernel_param_ops __param_ops_##name = \
|
||||
{ .flags = 0, (void *)set, (void *)get }; \
|
||||
{ .flags = 0, .set = _set, .get = _get }; \
|
||||
__module_param_call(MODULE_PARAM_PREFIX, \
|
||||
name, &__param_ops_##name, arg, perm, -1, 0)
|
||||
|
||||
|
|
Loading…
Reference in a new issue