]> git.ipfire.org Git - thirdparty/dbus.git/commitdiff
fdpass test: Assert that we don't leak file descriptors
authorSimon McVittie <smcv@collabora.com>
Thu, 16 Apr 2020 13:41:48 +0000 (14:41 +0100)
committerSimon McVittie <smcv@collabora.com>
Tue, 2 Jun 2020 09:42:54 +0000 (10:42 +0100)
Reproduces: dbus#294
Reproduces: CVE-2020-12049
Reproduces: GHSL-2020-057
Signed-off-by: Simon McVittie <smcv@collabora.com>
test/fdpass.c

index d84b7fe9912dfda08ad72725aea6acef2efcaded..abd415044250bea680174be5977cb1ce6f971413 100644 (file)
@@ -92,6 +92,7 @@ typedef struct {
     GQueue messages;
 
     int fd_before;
+    DBusInitialFDs *initial_fds;
 } Fixture;
 
 static void oom (const gchar *doing) G_GNUC_NORETURN;
@@ -176,6 +177,8 @@ test_connect (Fixture *f,
   if (f->skip)
     return;
 
+  f->initial_fds = _dbus_check_fdleaks_enter ();
+
   g_assert (f->left_server_conn == NULL);
   g_assert (f->right_server_conn == NULL);
 
@@ -875,6 +878,11 @@ teardown (Fixture *f,
   if (f->fd_before >= 0 && close (f->fd_before) < 0)
     g_error ("%s", g_strerror (errno));
 #endif
+
+  /* TODO: It would be nice if we could ask GLib which test-case
+   * we're currently in */
+  if (f->initial_fds != NULL)
+    _dbus_check_fdleaks_leave (f->initial_fds, "next test-case");
 }
 
 int