tests: check the return value of gpiod_line_config_add_line_settings()
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Thu, 13 Apr 2023 14:49:58 +0000 (16:49 +0200)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Thu, 13 Apr 2023 14:49:58 +0000 (16:49 +0200)
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>
tests/tests-info-event.c

index 9f942eb6a97ba0ef42623c5aa1dd03843e4251af..d069811482753ce1e471a38082e3fef3ef3b8b2a 100644 (file)
@@ -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);