diff options
author | H.J. Lu <hjl.tools@gmail.com> | 2020-02-01 05:43:34 -0800 |
---|---|---|
committer | H.J. Lu <hjl.tools@gmail.com> | 2020-02-01 05:43:34 -0800 |
commit | 635d6fae03257129b4672591b700a495cb6cb6c7 (patch) | |
tree | 6362ccb02eccc6303ab96a69c2db071aff01b0b2 | |
parent | d8775136b5830394d50c98997983b96a236629bf (diff) | |
download | glibc-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>
-rw-r--r-- | sysdeps/x86/tst-cet-legacy-5.c | 2 | ||||
-rw-r--r-- | sysdeps/x86/tst-cet-legacy-6.c | 2 |
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); } |