[PATCH] support: Enhance support/tst-support_descriptors compatibility

Florian Weimer fweimer@redhat.com
Thu Sep 5 09:03:42 GMT 2024


Some container environments have different devices for /dev and
/dev/null, leading to extra error output on the second line:

error: descriptor 3 changed from "/dev/null" to "/dev"
error: descriptor 3 changed device from 0:5 to 0:49
error: descriptor 3 changed ino from 4 to 1

The expected error matcher did not capture this before.
Technically, the inode number can remain unchanged due to the
changed device ID, so simply drop that check.

---
 support/tst-support_descriptors.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/support/tst-support_descriptors.c b/support/tst-support_descriptors.c
index 1e6e37a7fb..44ad152e21 100644
--- a/support/tst-support_descriptors.c
+++ b/support/tst-support_descriptors.c
@@ -130,9 +130,8 @@ test_run (void)
 
   expected = xasprintf ("\nDifferences:\n"
                         "error: descriptor %d changed from \"/dev/null\""
-                        " to \"/dev\"\n"
-                        "error: descriptor %d changed ino ",
-                        free_descriptor, free_descriptor);
+                        " to \"/dev\"\n",
+                        free_descriptor);
   good = good && !support_record_failure_is_failed ();
   proc = support_capture_subprocess (&subprocess_changed_descriptor, NULL);
   good = good && support_record_failure_is_failed ();

base-commit: ed416ee402dd0d014ed55b76c3b50c4700d875e3



More information about the Libc-alpha mailing list