Blame SOURCES/gdb-6.7-testsuite-stable-results.patch

b2f73e
From FEDORA_PATCHES Mon Sep 17 00:00:00 2001
b2f73e
From: Jan Kratochvil <jan.kratochvil@redhat.com>
b2f73e
Date: Fri, 27 Oct 2017 21:07:50 +0200
b2f73e
Subject: gdb-6.7-testsuite-stable-results.patch
b2f73e
b2f73e
;; Testsuite fixes for more stable/comparable results.
b2f73e
;;=fedoratest
b2f73e
b2f73e
gdb/testsuite/gdb.base/fileio.c:
b2f73e
gdb/testsuite/gdb.base/fileio.exp:
b2f73e
2007-12-08  Jan Kratochvil  <jan.kratochvil@redhat.com>
b2f73e
b2f73e
	* gdb.base/fileio.c (ROOTSUBDIR): New macro.
b2f73e
	(main): CHDIR into ROOTSUBDIR.  CHOWN ROOTSUBDIR and CHDIR into
b2f73e
	ROOTSUBDIR if we are being run as root.
b2f73e
	* gdb.base/fileio.exp: Change the startup and finish cleanup.
b2f73e
	Change the test file reference to be into the `fileio.dir' directory.
b2f73e
b2f73e
sources/gdb/testsuite/gdb.base/dump.exp:
b2f73e
Found on RHEL-5.s390x.
b2f73e
b2f73e
gdb-6.8.50.20090209/gdb/testsuite/gdb.base/auxv.exp:
b2f73e
random FAIL: gdb.base/auxv.exp: matching auxv data from live and gcore
b2f73e
b2f73e
gdb-6.8.50.20090209/gdb/testsuite/gdb.base/annota1.exp:
b2f73e
frames-invalid can happen asynchronously.
b2f73e
b2f73e
diff --git a/gdb/testsuite/gdb.base/fileio.c b/gdb/testsuite/gdb.base/fileio.c
b2f73e
--- a/gdb/testsuite/gdb.base/fileio.c
b2f73e
+++ b/gdb/testsuite/gdb.base/fileio.c
b2f73e
@@ -560,6 +560,28 @@ strerrno (int err)
b2f73e
 int
b2f73e
 main ()
b2f73e
 {
b2f73e
+  /* These tests
b2f73e
+       Open for write but no write permission returns EACCES
b2f73e
+       Unlinking a file in a directory w/o write access returns EACCES
b2f73e
+     fail if we are being run as root - drop the privileges here.  */
b2f73e
+
b2f73e
+  if (geteuid () == 0)
b2f73e
+    {
b2f73e
+      uid_t uid = 99;
b2f73e
+
b2f73e
+      if (chown (OUTDIR, uid, uid) != 0)
b2f73e
+	{
b2f73e
+	  printf ("chown %d.%d %s: %s\n", (int) uid, (int) uid,
b2f73e
+		  OUTDIR, strerror (errno));
b2f73e
+	  exit (1);
b2f73e
+	}
b2f73e
+      if (setuid (uid) || geteuid () == 0)
b2f73e
+	{
b2f73e
+	  printf ("setuid %d: %s\n", (int) uid, strerror (errno));
b2f73e
+	  exit (1);
b2f73e
+	}
b2f73e
+    }
b2f73e
+
b2f73e
   /* Don't change the order of the calls.  They partly depend on each other */
b2f73e
   test_open ();
b2f73e
   test_write ();
b2f73e
diff --git a/gdb/testsuite/gdb.base/fileio.exp b/gdb/testsuite/gdb.base/fileio.exp
b2f73e
--- a/gdb/testsuite/gdb.base/fileio.exp
b2f73e
+++ b/gdb/testsuite/gdb.base/fileio.exp
b2f73e
@@ -24,9 +24,9 @@ if [target_info exists gdb,nofileio] {
b2f73e
 standard_testfile
b2f73e
 
b2f73e
 if {[is_remote host]} {
b2f73e
-    set outdir .
b2f73e
+    set outdir "fileio.dir"
b2f73e
 } else {
b2f73e
-    set outdir [standard_output_file {}]
b2f73e
+    set outdir [standard_output_file "fileio.dir"]
b2f73e
 }
b2f73e
 
b2f73e
 if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" \
b2f73e
@@ -47,7 +47,8 @@ set dir2 [standard_output_file dir2.fileio.test]
b2f73e
 if {[file exists $dir2] && ![file writable $dir2]} {
b2f73e
     system "chmod +w $dir2"
b2f73e
 }
b2f73e
-system "rm -rf [standard_output_file *.fileio.test]"
b2f73e
+system "rm -rf [standard_output_file fileio.dir]"
b2f73e
+system "mkdir -m777 [standard_output_file fileio.dir]"
b2f73e
 
b2f73e
 set oldtimeout $timeout
b2f73e
 set timeout [expr "$timeout + 60"]
b2f73e
@@ -89,7 +90,7 @@ gdb_test continue \
b2f73e
 
b2f73e
 gdb_test "continue" ".*" ""
b2f73e
 
b2f73e
-catch "system \"chmod -f -w [standard_output_file nowrt.fileio.test]\""
b2f73e
+catch "system \"chmod -f -w [standard_output_file fileio.dir/nowrt.fileio.test]\""
b2f73e
 
b2f73e
 gdb_test continue \
b2f73e
 "Continuing\\..*open 5:.*EACCES$stop_msg" \
b2f73e
@@ -276,9 +277,7 @@ gdb_test continue \
b2f73e
 gdb_exit
b2f73e
 
b2f73e
 # Make dir2 writable again so rm -rf of a build tree Just Works.
b2f73e
-if {[file exists $dir2] && ![file writable $dir2]} {
b2f73e
-    system "chmod +w $dir2"
b2f73e
-}
b2f73e
+system "chmod -R +w $outdir"
b2f73e
 
b2f73e
 set timeout $oldtimeout
b2f73e
 return 0