about summary refs log tree commit diff
path: root/sysdeps/x86
diff options
context:
space:
mode:
authorH.J. Lu <hjl.tools@gmail.com>2020-02-01 05:43:34 -0800
committerH.J. Lu <hjl.tools@gmail.com>2020-02-01 05:43:34 -0800
commit635d6fae03257129b4672591b700a495cb6cb6c7 (patch)
tree6362ccb02eccc6303ab96a69c2db071aff01b0b2 /sysdeps/x86
parentd8775136b5830394d50c98997983b96a236629bf (diff)
downloadglibc-635d6fae03257129b4672591b700a495cb6cb6c7.tar.gz
glibc-635d6fae03257129b4672591b700a495cb6cb6c7.tar.xz
glibc-635d6fae03257129b4672591b700a495cb6cb6c7.zip
x86: Don't make 2 calls to dlerror () in a row
We shouldn't make 2 calls to dlerror () in a row since the first call
will clear the error.  We should just use the return value from the
first call.

Tested on Linux/x86-64.

Reviewed-by: Florian Weimer <fweimer@redhat.com>
Diffstat (limited to 'sysdeps/x86')
-rw-r--r--sysdeps/x86/tst-cet-legacy-5.c2
-rw-r--r--sysdeps/x86/tst-cet-legacy-6.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/sysdeps/x86/tst-cet-legacy-5.c b/sysdeps/x86/tst-cet-legacy-5.c
index b97e03d634..6c9bba06f5 100644
--- a/sysdeps/x86/tst-cet-legacy-5.c
+++ b/sysdeps/x86/tst-cet-legacy-5.c
@@ -38,7 +38,7 @@ do_test_1 (const char *modname, bool fail)
 	  if (strstr (err, "shadow stack isn't enabled") == NULL)
 	    {
 	      printf ("incorrect dlopen '%s' error: %s\n", modname,
-		      dlerror ());
+		      err);
 	      exit (1);
 	    }
 
diff --git a/sysdeps/x86/tst-cet-legacy-6.c b/sysdeps/x86/tst-cet-legacy-6.c
index 3c4a47f2db..877e77747d 100644
--- a/sysdeps/x86/tst-cet-legacy-6.c
+++ b/sysdeps/x86/tst-cet-legacy-6.c
@@ -38,7 +38,7 @@ do_test_1 (const char *modname, bool fail)
 	  if (strstr (err, "shadow stack isn't enabled") == NULL)
 	    {
 	      printf ("incorrect dlopen '%s' error: %s\n", modname,
-		      dlerror ());
+		      err);
 	      exit (1);
 	    }