selftests: uevent filtering: fix return on error in uevent_listener
authorJavier Carrasco <javier.carrasco.cruz@gmail.com>
Sun, 1 Oct 2023 08:37:44 +0000 (10:37 +0200)
committerShuah Khan <skhan@linuxfoundation.org>
Thu, 5 Oct 2023 20:28:53 +0000 (14:28 -0600)
commit5b45a753776be5d21cf395ec97e81c9187fbeaca
tree66fb1e4b4910d54e9046890921e01a89c345868a
parent37013b557b7f39e603b4459ef82dc8b848f98a67
selftests: uevent filtering: fix return on error in uevent_listener

The ret variable is used to check function return values and assigning
values to it on error has no effect as it is an unused value.

The current implementation uses an additional variable (fret) to return
the error value, which in this case is unnecessary and lead to the above
described misuse. There is no restriction in the current implementation
to always return -1 on error and the actual negative error value can be
returned safely without storing -1 in a specific variable.

Simplify the error checking by using a single variable which always
holds the returned value.

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@gmail.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
tools/testing/selftests/uevent/uevent_filtering.c