counter: 104-quad-8: Fix race getting function mode and direction
authorWilliam Breathitt Gray <william.gray@linaro.org>
Thu, 20 Oct 2022 14:11:21 +0000 (10:11 -0400)
committerWilliam Breathitt Gray <william.gray@linaro.org>
Mon, 24 Oct 2022 00:39:26 +0000 (20:39 -0400)
commitd501d37841d3b7f18402d71a9ef057eb9dde127e
tree7614710f955f636f75c0b75e021de8af08f1fac9
parentd917a62af81b133f35f627e7936e193c842a7947
counter: 104-quad-8: Fix race getting function mode and direction

The quad8_action_read() function checks the Count function mode and
Count direction without first acquiring a lock. This is a race condition
because the function mode could change by the time the direction is
checked.

Because the quad8_function_read() already acquires a lock internally,
the quad8_function_read() is refactored to spin out the no-lock code to
a new quad8_function_get() function.

To resolve the race condition in quad8_action_read(), a lock is acquired
before calling quad8_function_get() and quad8_direction_read() in order
to get both function mode and direction atomically.

Fixes: f1d8a071d45b ("counter: 104-quad-8: Add Generic Counter interface support")
Cc: stable@vger.kernel.org
Link: https://lore.kernel.org/r/20221020141121.15434-1-william.gray@linaro.org/
Signed-off-by: William Breathitt Gray <william.gray@linaro.org>
drivers/counter/104-quad-8.c