From fffa1cf8a7f95830d488d5fc6bf73cb9599143f4 Mon Sep 17 00:00:00 2001 From: Leonhard Holz Date: Wed, 12 Nov 2014 17:10:21 +0530 Subject: Fix tst-strcoll-overflow returning before timeout (BZ #17506) Modifies the test examination in test-skeleton.c so that a test can be successful if it is interrupted or it returns uninterrupted with the expected status. For this both EXPECTED_SIGNAL and EXPECTED_STATUS have to be set, as is done in tst-strcoll-overflow.c. --- ChangeLog | 7 +++++ NEWS | 3 ++- string/tst-strcoll-overflow.c | 1 + test-skeleton.c | 63 ++++++++++++++++++++++++------------------- 4 files changed, 45 insertions(+), 29 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6a7f43549e..a7d3298270 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2014-11-12 Leonhard Holz + + [BZ #17506] + * test-skeleton.c (main): Return successful if one of + EXPECTED_SIGNAL or EXPECTED_STATUS is met when both given. + * string/tst-strcoll-overflow.c: Define expected status. + 2014-11-12 Tatiana Udalova [BZ #17475] diff --git a/NEWS b/NEWS index df35b808e1..a1743d7ecf 100644 --- a/NEWS +++ b/NEWS @@ -10,7 +10,8 @@ Version 2.21 * The following bugs are resolved with this release: 6652, 12926, 14132, 14138, 14171, 15215, 15884, 17266, 17344, 17363, - 17370, 17371, 17411, 17460, 17475, 17485, 17501, 17508, 17522, 17555. + 17370, 17371, 17411, 17460, 17475, 17485, 17501, 17506, 17508, 17522, + 17555. * New locales: tu_IN, bh_IN. diff --git a/string/tst-strcoll-overflow.c b/string/tst-strcoll-overflow.c index e7a43ea844..0d764afff6 100644 --- a/string/tst-strcoll-overflow.c +++ b/string/tst-strcoll-overflow.c @@ -57,5 +57,6 @@ do_test (void) #define TIMEOUT 300 #define EXPECTED_SIGNAL SIGALRM +#define EXPECTED_STATUS 0 #define TEST_FUNCTION do_test () #include "../test-skeleton.c" diff --git a/test-skeleton.c b/test-skeleton.c index c1278ca3b2..3621009b7e 100644 --- a/test-skeleton.c +++ b/test-skeleton.c @@ -383,39 +383,46 @@ main (int argc, char *argv[]) exit (1); } -#ifndef EXPECTED_SIGNAL - /* We don't expect any signal. */ -# define EXPECTED_SIGNAL 0 -#endif - if (WTERMSIG (status) != EXPECTED_SIGNAL) + /* Process terminated normaly without timeout etc. */ + if (WIFEXITED (status)) { - if (EXPECTED_SIGNAL != 0) - { - if (WTERMSIG (status) == 0) - printf ("Expected signal '%s' from child, got none\n", - strsignal (EXPECTED_SIGNAL)); - else - printf ("Incorrect signal from child: got `%s', need `%s'\n", - strsignal (WTERMSIG (status)), - strsignal (EXPECTED_SIGNAL)); - } - else - printf ("Didn't expect signal from child: got `%s'\n", - strsignal (WTERMSIG (status))); - exit (1); - } - - /* Simply exit with the return value of the test. */ #ifndef EXPECTED_STATUS - return WEXITSTATUS (status); +# ifndef EXPECTED_SIGNAL + /* Simply exit with the return value of the test. */ + return WEXITSTATUS (status); +# else + printf ("Expected signal '%s' from child, got none\n", + strsignal (EXPECTED_SIGNAL)); + exit (1); +# endif #else - if (WEXITSTATUS (status) != EXPECTED_STATUS) + if (WEXITSTATUS (status) != EXPECTED_STATUS) + { + printf ("Expected status %d, got %d\n", + EXPECTED_STATUS, WEXITSTATUS (status)); + exit (1); + } + + return 0; +#endif + } + /* Process was killed by timer or other signal. */ + else { - printf ("Expected status %d, got %d\n", - EXPECTED_STATUS, WEXITSTATUS (status)); +#ifndef EXPECTED_SIGNAL + printf ("Didn't expect signal from child: got `%s'\n", + strsignal (WTERMSIG (status))); exit (1); - } +#else + if (WTERMSIG (status) != EXPECTED_SIGNAL) + { + printf ("Incorrect signal from child: got `%s', need `%s'\n", + strsignal (WTERMSIG (status)), + strsignal (EXPECTED_SIGNAL)); + exit (1); + } - return 0; + return 0; #endif + } } -- cgit v1.2.3