[v2] tst-getdate: Improve testcase flexibility and add test.
Checks
Context |
Check |
Description |
redhat-pt-bot/TryBot-apply_patch |
success
|
Patch applied to master at the time it was sent
|
redhat-pt-bot/TryBot-32bit |
success
|
Build for i686
|
linaro-tcwg-bot/tcwg_glibc_check--master-arm |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_glibc_build--master-aarch64 |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_glibc_build--master-arm |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_glibc_check--master-aarch64 |
success
|
Testing passed
|
Commit Message
The getdate testcases all expect successful results. Add support for
negative testcases and testcases where a full date and time are not
supplied by skipping the tm checks in the test. Add a testcase that
would catch a use-after-free that was recently found.
---
Changes to v1:
* Retain old testcase and add a new one with spaces in the middle of
the string.
* Move the logic for getdate() failure into the same area.
* Use TEST_VERIFY to compare pointer to NULL rather than
TEST_COMPARE_BLOB with size 0 which is a nop.
time/tst-getdate.c | 62 +++++++++++++++++++++++++++++++---------------
1 file changed, 42 insertions(+), 20 deletions(-)
Comments
> The getdate testcases all expect successful results. Add support for
> negative testcases and testcases where a full date and time are not
> supplied by skipping the tm checks in the test. Add a testcase that
> would catch a use-after-free that was recently found.
Thanks for the v2! This LGTM with two nitpicks below.
Reviewed-by: Arjun Shankar <arjun@redhat.com>
> ---
> Changes to v1:
> * Retain old testcase and add a new one with spaces in the middle of
> the string.
> * Move the logic for getdate() failure into the same area.
> * Use TEST_VERIFY to compare pointer to NULL rather than
> TEST_COMPARE_BLOB with size 0 which is a nop.
>
> time/tst-getdate.c | 62 +++++++++++++++++++++++++++++++---------------
> 1 file changed, 42 insertions(+), 20 deletions(-)
>
> diff --git a/time/tst-getdate.c b/time/tst-getdate.c
> index 4c9ed28d58..78486ca2ed 100644
> --- a/time/tst-getdate.c
> +++ b/time/tst-getdate.c
> @@ -32,34 +32,42 @@ static const struct
> const char *tz;
> struct tm tm;
> bool time64;
> + int err_val;
> + bool check_tm;
> } tests [] =
> {
> {"21:01:10 1999-1-31", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
> - false },
> + false , 0, true},
> {"21:01:10 1999-1-31", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
> - false },
> + false , 0, true},
> {" 21:01:10 1999-1-31", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
> - false },
> + false , 0, true},
> {"21:01:10 1999-1-31 ", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
> - false },
> + false , 0, true},
> {" 21:01:10 1999-1-31 ", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
> - false },
> + false , 0, true},
> {"21:01:10 1999-2-28", "Universal", {10, 1, 21, 28, 1, 99, 0, 0, 0},
> - false },
> + false , 0, true},
> {"16:30:46 2000-2-29", "Universal", {46, 30,16, 29, 1, 100, 0, 0, 0},
> - false },
> + false , 0, true},
> {"01-08-2000 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
> - false },
> + false , 0, true},
> + {"01-08-2000 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
> + false , 0, true},
> + {"01-08-2000 a 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
> + false , 7, false},
> + {" 12 AM ", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
> + false , 0, false},
Let's set the tm struct fields to 0 here for tests that don't have tm checking?
>
> /* 64 bit time_t tests. */
> {"21:01:10 2038-1-31", "Universal", {10, 1, 21, 31, 0, 138, 0, 0, 0},
> - true },
> + true , 0, true},
> {"22:01:10 2048-5-20", "Universal", {10, 1, 22, 20, 4, 148, 0, 0, 0},
> - true },
> + true , 0, true},
> {"01-08-2038 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 138, 0, 0, 0},
> - true },
> + true , 0, true},
> {"20-03-2050 21:30:08", "Europe/Berlin", {8, 30, 21, 20, 2, 150, 0, 0, 0},
> - true }
> + true , 0, true}
> };
>
> static const char *
> @@ -93,7 +101,8 @@ report_date_error (void)
> static char *datemsk;
> static const char datemskstr[] =
> "%H:%M:%S %F\n"
> - "%d-%m-%Y %T\n";
> + "%d-%m-%Y %T\n"
> + "%I %p\n";
>
> static void
> do_prepare (int argc, char **argv)
> @@ -115,13 +124,25 @@ do_test (void)
> setenv ("TZ", tests[i].tz, 1);
>
> tm = getdate (tests[i].str);
> - TEST_COMPARE (getdate_err, 0);
> - if (getdate_err != 0)
> +
> + /* Only check getdate_err when tm is NULL as getdate doesn't set
> + getdate_err on success. */
> + if (tm == NULL)
> + {
> + TEST_COMPARE (getdate_err, tests[i].err_val);
> + if (getdate_err != tests[i].err_val)
> + {
> + support_record_failure ();
The call to support_record_failure is redundant because TEST_COMPARE
also calls it. We could get rid of it now that we're updating this
test.
> + printf ("%s\n", report_date_error ());
> + }
> + }
> + if (tests[i].err_val != 0) /* Expected failure */
> {
> - support_record_failure ();
> - printf ("%s\n", report_date_error ());
> + TEST_VERIFY (tm == NULL);
> + continue;
> }
> - else
> +
> + if (tests[i].check_tm)
> {
> TEST_COMPARE (tests[i].tm.tm_mon, tm->tm_mon);
> TEST_COMPARE (tests[i].tm.tm_year, tm->tm_year);
> @@ -132,8 +153,9 @@ do_test (void)
> }
>
> struct tm tms;
> - TEST_COMPARE (getdate_r (tests[i].str, &tms), 0);
> - if (getdate_err == 0)
> + int retval = getdate_r (tests[i].str, &tms);
> + TEST_COMPARE (retval, tests[i].err_val);
> + if (retval == tests[i].err_val && tests[i].check_tm)
> {
> TEST_COMPARE (tests[i].tm.tm_mon, tms.tm_mon);
> TEST_COMPARE (tests[i].tm.tm_year, tms.tm_year);
> --
> 2.39.2
>
@@ -32,34 +32,42 @@ static const struct
const char *tz;
struct tm tm;
bool time64;
+ int err_val;
+ bool check_tm;
} tests [] =
{
{"21:01:10 1999-1-31", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
- false },
+ false , 0, true},
{"21:01:10 1999-1-31", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
- false },
+ false , 0, true},
{" 21:01:10 1999-1-31", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
- false },
+ false , 0, true},
{"21:01:10 1999-1-31 ", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
- false },
+ false , 0, true},
{" 21:01:10 1999-1-31 ", "Universal", {10, 1, 21, 31, 0, 99, 0, 0, 0},
- false },
+ false , 0, true},
{"21:01:10 1999-2-28", "Universal", {10, 1, 21, 28, 1, 99, 0, 0, 0},
- false },
+ false , 0, true},
{"16:30:46 2000-2-29", "Universal", {46, 30,16, 29, 1, 100, 0, 0, 0},
- false },
+ false , 0, true},
{"01-08-2000 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
- false },
+ false , 0, true},
+ {"01-08-2000 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
+ false , 0, true},
+ {"01-08-2000 a 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
+ false , 7, false},
+ {" 12 AM ", "Europe/Berlin", {7, 6, 5, 1, 7, 100, 0, 0, 0},
+ false , 0, false},
/* 64 bit time_t tests. */
{"21:01:10 2038-1-31", "Universal", {10, 1, 21, 31, 0, 138, 0, 0, 0},
- true },
+ true , 0, true},
{"22:01:10 2048-5-20", "Universal", {10, 1, 22, 20, 4, 148, 0, 0, 0},
- true },
+ true , 0, true},
{"01-08-2038 05:06:07", "Europe/Berlin", {7, 6, 5, 1, 7, 138, 0, 0, 0},
- true },
+ true , 0, true},
{"20-03-2050 21:30:08", "Europe/Berlin", {8, 30, 21, 20, 2, 150, 0, 0, 0},
- true }
+ true , 0, true}
};
static const char *
@@ -93,7 +101,8 @@ report_date_error (void)
static char *datemsk;
static const char datemskstr[] =
"%H:%M:%S %F\n"
- "%d-%m-%Y %T\n";
+ "%d-%m-%Y %T\n"
+ "%I %p\n";
static void
do_prepare (int argc, char **argv)
@@ -115,13 +124,25 @@ do_test (void)
setenv ("TZ", tests[i].tz, 1);
tm = getdate (tests[i].str);
- TEST_COMPARE (getdate_err, 0);
- if (getdate_err != 0)
+
+ /* Only check getdate_err when tm is NULL as getdate doesn't set
+ getdate_err on success. */
+ if (tm == NULL)
+ {
+ TEST_COMPARE (getdate_err, tests[i].err_val);
+ if (getdate_err != tests[i].err_val)
+ {
+ support_record_failure ();
+ printf ("%s\n", report_date_error ());
+ }
+ }
+ if (tests[i].err_val != 0) /* Expected failure */
{
- support_record_failure ();
- printf ("%s\n", report_date_error ());
+ TEST_VERIFY (tm == NULL);
+ continue;
}
- else
+
+ if (tests[i].check_tm)
{
TEST_COMPARE (tests[i].tm.tm_mon, tm->tm_mon);
TEST_COMPARE (tests[i].tm.tm_year, tm->tm_year);
@@ -132,8 +153,9 @@ do_test (void)
}
struct tm tms;
- TEST_COMPARE (getdate_r (tests[i].str, &tms), 0);
- if (getdate_err == 0)
+ int retval = getdate_r (tests[i].str, &tms);
+ TEST_COMPARE (retval, tests[i].err_val);
+ if (retval == tests[i].err_val && tests[i].check_tm)
{
TEST_COMPARE (tests[i].tm.tm_mon, tms.tm_mon);
TEST_COMPARE (tests[i].tm.tm_year, tms.tm_year);