diff options
author | Kees Cook <keescook@chromium.org> | 2013-07-03 18:04:58 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-07-03 19:07:41 -0400 |
commit | f170168b9a0b61ea1e647b082b38f605f1d3de3e (patch) | |
tree | 4381c129ee9af108e368e6c8255df53cbe233257 /drivers/hwmon/adt7470.c | |
parent | d8537548c924db3c44afde7646b6e220c7beb79d (diff) |
drivers: avoid parsing names as kthread_run() format strings
Calling kthread_run with a single name parameter causes it to be handled
as a format string. Many callers are passing potentially dynamic string
content, so use "%s" in those cases to avoid any potential accidents.
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/hwmon/adt7470.c')
-rw-r--r-- | drivers/hwmon/adt7470.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/adt7470.c b/drivers/hwmon/adt7470.c index b83bf4bb95eb..0f34bca9f5e5 100644 --- a/drivers/hwmon/adt7470.c +++ b/drivers/hwmon/adt7470.c | |||
@@ -1285,7 +1285,7 @@ static int adt7470_probe(struct i2c_client *client, | |||
1285 | } | 1285 | } |
1286 | 1286 | ||
1287 | init_completion(&data->auto_update_stop); | 1287 | init_completion(&data->auto_update_stop); |
1288 | data->auto_update = kthread_run(adt7470_update_thread, client, | 1288 | data->auto_update = kthread_run(adt7470_update_thread, client, "%s", |
1289 | dev_name(data->hwmon_dev)); | 1289 | dev_name(data->hwmon_dev)); |
1290 | if (IS_ERR(data->auto_update)) { | 1290 | if (IS_ERR(data->auto_update)) { |
1291 | err = PTR_ERR(data->auto_update); | 1291 | err = PTR_ERR(data->auto_update); |