On Sat, May 27, 2017 at 7:45 PM, Andy Shevchenko <[email protected]> wrote: > On Thu, May 25, 2017 at 11:33 AM, Bartosz Golaszewski <[email protected]> wrote: >> We're currently only checking the first character of the input to the >> debugfs event files, so a string like '0sdfdsf' is valid and indicates >> a falling edge event. >> >> Be more strict and only allow '0', '1', '0\n' & '1\n'. > > Why not to be so strict and use > > kstrtobool_from_user(); > > instead?
Or if you still wish to be strict, ret = kstrtou8_from_user(); if (ret) return ret; if (val > 1) return -ERANGE; or alike. -- With Best Regards, Andy Shevchenko
