Use kstrtobool in trace_remote.c where possible. This is more user-friendly as it allows a better variety of input strings.
Signed-off-by: Vincent Donnefort <[email protected]> diff --git a/kernel/trace/trace_remote.c b/kernel/trace/trace_remote.c index 9b27c7bd6040..71f6cda0fbd4 100644 --- a/kernel/trace/trace_remote.c +++ b/kernel/trace/trace_remote.c @@ -235,10 +235,10 @@ tracing_on_write(struct file *filp, const char __user *ubuf, size_t cnt, loff_t { struct seq_file *seq = filp->private_data; struct trace_remote *remote = seq->private; - unsigned long val; + bool val; int ret; - ret = kstrtoul_from_user(ubuf, cnt, 10, &val); + ret = kstrtobool_from_user(ubuf, cnt, &val); if (ret) return ret; @@ -1154,10 +1154,10 @@ static ssize_t remote_event_enable_write(struct file *filp, const char __user *u struct seq_file *seq = filp->private_data; struct remote_event *evt = seq->private; struct trace_remote *remote = evt->remote; - u8 enable; + bool enable; int ret; - ret = kstrtou8_from_user(ubuf, count, 10, &enable); + ret = kstrtobool_from_user(ubuf, count, &enable); if (ret) return ret; @@ -1238,10 +1238,10 @@ static ssize_t remote_events_dir_enable_write(struct file *filp, const char __us size_t count, loff_t *ppos) { struct trace_remote *remote = file_inode(filp)->i_private; + bool enable; int i, ret; - u8 enable; - ret = kstrtou8_from_user(ubuf, count, 10, &enable); + ret = kstrtobool_from_user(ubuf, count, &enable); if (ret) return ret; -- 2.54.0.1032.g2f8565e1d1-goog
