[v2] Input: evdev/uinput errno alignment

[PATCH v2 0/2] Input: evdev/uinput errno alignment

Iván Ezequiel Rodriguez posted 2 patches 3 weeks, 4 days ago

Diff against v1

Only 0 patches received!

drivers/input/evdev.c       | 5 +++--
drivers/input/misc/uinput.c | 2 +-
2 files changed, 4 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH v2 0/2] Input: evdev/uinput errno alignment

Posted by Iván Ezequiel Rodriguez 3 weeks, 4 days ago

Two small fixes making evdev and uinput report the error codes their
uapi documentation promises.

Changes since v1:
 - Dropped the ABS_MT_SLOT "minimum must be 0" validation from patch 2.
   It was a userspace regression: uinput_create_device() already calls
   input_mt_init_slots() for devices declaring ABS_MT_SLOT, and
   input_mt_init_slots() overwrites the axis with
   input_set_abs_params(dev, ABS_MT_SLOT, 0, num_slots - 1, 0, 0).
   A non-zero minimum is therefore accepted today and silently
   corrected, so rejecting it with -EINVAL would break existing
   callers.
 - Patch 2 is now limited to the -ERANGE to -EINVAL change and its
   commit message was reworded accordingly.
 - Patch 1 is unchanged.

v1: https://lore.kernel.org/linux-input/20260831152156.166514-1-ivanrwcm25@gmail.com/

Iván Ezequiel Rodriguez (2):
  Input: evdev: propagate EVIOCSFF copy errors correctly
  Input: uinput: return -EINVAL for out-of-range UI_ABS_SETUP axis code

 drivers/input/evdev.c       | 5 +++--
 drivers/input/misc/uinput.c | 2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

-- 
2.43.0