diff options
Diffstat (limited to 'kernel/params.c')
-rw-r--r-- | kernel/params.c | 17 |
1 files changed, 16 insertions, 1 deletions
diff --git a/kernel/params.c b/kernel/params.c index a550698ae02d..458a09b886c4 100644 --- a/kernel/params.c +++ b/kernel/params.c | |||
@@ -399,9 +399,20 @@ static int param_array_get(char *buffer, const struct kernel_param *kp) | |||
399 | return off; | 399 | return off; |
400 | } | 400 | } |
401 | 401 | ||
402 | static void param_array_free(void *arg) | ||
403 | { | ||
404 | unsigned int i; | ||
405 | const struct kparam_array *arr = arg; | ||
406 | |||
407 | if (arr->ops->free) | ||
408 | for (i = 0; i < (arr->num ? *arr->num : arr->max); i++) | ||
409 | arr->ops->free(arr->elem + arr->elemsize * i); | ||
410 | } | ||
411 | |||
402 | struct kernel_param_ops param_array_ops = { | 412 | struct kernel_param_ops param_array_ops = { |
403 | .set = param_array_set, | 413 | .set = param_array_set, |
404 | .get = param_array_get, | 414 | .get = param_array_get, |
415 | .free = param_array_free, | ||
405 | }; | 416 | }; |
406 | EXPORT_SYMBOL(param_array_ops); | 417 | EXPORT_SYMBOL(param_array_ops); |
407 | 418 | ||
@@ -634,7 +645,11 @@ void module_param_sysfs_remove(struct module *mod) | |||
634 | 645 | ||
635 | void destroy_params(const struct kernel_param *params, unsigned num) | 646 | void destroy_params(const struct kernel_param *params, unsigned num) |
636 | { | 647 | { |
637 | /* FIXME: This should free kmalloced charp parameters. It doesn't. */ | 648 | unsigned int i; |
649 | |||
650 | for (i = 0; i < num; i++) | ||
651 | if (params[i].ops->free) | ||
652 | params[i].ops->free(params[i].arg); | ||
638 | } | 653 | } |
639 | 654 | ||
640 | static void __init kernel_add_sysfs_param(const char *name, | 655 | static void __init kernel_add_sysfs_param(const char *name, |