diff options
author | Bartosz Golaszewski <bartosz.golaszewski@linaro.org> | 2023-04-13 16:49:58 +0200 |
---|---|---|
committer | Bartosz Golaszewski <bartosz.golaszewski@linaro.org> | 2023-04-13 16:49:58 +0200 |
commit | 010207cd0e586ddea4d805deeffb3df4b578eebe (patch) | |
tree | 6c432b783df82a9e3814907bfac6f59e03dbeff6 | |
parent | 0929f3990edda0c84ce4c82156c94989580e88a3 (diff) | |
download | libgpiod-010207cd0e586ddea4d805deeffb3df4b578eebe.tar.gz |
tests: check the return value of gpiod_line_config_add_line_settings()
The call to gpiod_line_config_add_line_settings() can fail in the thread
function used by info-event test cases so check its return value and act
accordingly.
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
-rw-r--r-- | tests/tests-info-event.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/tests/tests-info-event.c b/tests/tests-info-event.c index 9f942eb6..d0698114 100644 --- a/tests/tests-info-event.c +++ b/tests/tests-info-event.c @@ -79,8 +79,11 @@ static gpointer request_reconfigure_release_line(gpointer data) gpiod_line_config_reset(ctx->line_cfg); gpiod_line_settings_set_direction(ctx->settings, GPIOD_LINE_DIRECTION_OUTPUT); - gpiod_line_config_add_line_settings(ctx->line_cfg, &ctx->offset, 1, - ctx->settings); + ret = gpiod_line_config_add_line_settings(ctx->line_cfg, &ctx->offset, + 1, ctx->settings); + g_assert_cmpint(ret, ==, 0); + if (g_test_failed()) + return NULL; ret = gpiod_line_request_reconfigure_lines(request, ctx->line_cfg); g_assert_cmpint(ret, ==, 0); |