At the moment we're leaking the GEM and FB objects that could lead to
OOM if multiple subtests are run (vs. each subtest in a seperate test
run).

Signed-off-by: Imre Deak <redacted>
---
  tests/kms_ccs.c | 15 +++++++++++----
  1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index 586680ae1..f376f1c80 100644
--- a/tests/kms_ccs.c
+++ b/tests/kms_ccs.c

@@ -345,6 +345,10 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
  	if (data->flags & TEST_FAIL_ON_ADDFB2) {
  		igt_assert_eq(ret, -1);
  		igt_assert_eq(errno, EINVAL);
+
+		if (f.handles[index])
+			gem_close(data->drm_fd, f.handles[index]);
+

  		return;
  	} else
  		igt_assert_eq(ret, 0);

@@ -387,7 +391,8 @@ static bool try_config(data_t *data, enum test_fb_flags fb_flags,
  	drmModeModeInfo *drm_mode = igt_output_get_mode(data->output);
  	int fb_width = drm_mode->hdisplay;
  	enum igt_commit_style commit;
-	struct igt_fb fb, fb_sprite;
+	struct igt_fb fb = {};
+	struct igt_fb fb_sprite = {};
  	int ret;
  
  	if (data->display.is_atomic)

@@ -425,7 +430,7 @@ static bool try_config(data_t *data, enum test_fb_flags fb_flags,
  	}
  
  	if (data->flags & TEST_FAIL_ON_ADDFB2)
-		return true;
+		goto out_free_fbs;
  
  	igt_plane_set_position(primary, 0, 0);
  	igt_plane_set_size(primary, drm_mode->hdisplay, drm_mode->vdisplay);

@@ -458,14 +463,16 @@ static bool try_config(data_t *data, enum test_fb_flags fb_flags,
  		igt_plane_set_position(data->plane, 0, 0);
  		igt_plane_set_size(data->plane, 0, 0);
  		igt_plane_set_fb(data->plane, NULL);
-		igt_remove_fb(display->drm_fd, &fb_sprite);
  	}
  
  	igt_plane_set_fb(primary, NULL);
  	igt_plane_set_rotation(primary, IGT_ROTATION_0);
  	igt_display_commit2(display, commit);
  
-	if (data->flags & TEST_CRC)
+out_free_fbs:
+	if (fb_sprite.fb_id)
+		igt_remove_fb(data->drm_fd, &fb_sprite);
+	if (fb.fb_id)
  		igt_remove_fb(data->drm_fd, &fb);