diff mbox series

tst: Add test for futimens

Message ID 20210225003313.6847-1-lukma@denx.de
State Committed
Delegated to: Adhemerval Zanella Netto
Headers show
Series tst: Add test for futimens | expand

Commit Message

Lukasz Majewski Feb. 25, 2021, 12:33 a.m. UTC
This patch provides test for futimens. It uses xfstat() syscall to read
access and modification time to compare with ones written by futimens.

Moreover, access and modification times beyond the Y2038 threshold
date (i.e. 32 bit time_t overflow) are also checked.
---
 sysdeps/unix/sysv/linux/Makefile       |  2 +-
 sysdeps/unix/sysv/linux/tst-futimens.c | 80 ++++++++++++++++++++++++++
 2 files changed, 81 insertions(+), 1 deletion(-)
 create mode 100644 sysdeps/unix/sysv/linux/tst-futimens.c

Comments

Adhemerval Zanella Feb. 25, 2021, 12:04 p.m. UTC | #1
On 24/02/2021 21:33, Lukasz Majewski wrote:
> This patch provides test for futimens. It uses xfstat() syscall to read

The xfstat is a wrapper, maybe just describe it access the access and
modification time to compare the one returned by futimens (without
describing the syscall used for).

> access and modification time to compare with ones written by futimens.
> 
> Moreover, access and modification times beyond the Y2038 threshold
> date (i.e. 32 bit time_t overflow) are also checked.

LGTM with the commit message change the small nit below.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>

> ---
>  sysdeps/unix/sysv/linux/Makefile       |  2 +-
>  sysdeps/unix/sysv/linux/tst-futimens.c | 80 ++++++++++++++++++++++++++
>  2 files changed, 81 insertions(+), 1 deletion(-)
>  create mode 100644 sysdeps/unix/sysv/linux/tst-futimens.c
> 
> diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile
> index eca24cdfe8..4e043c78ab 100644
> --- a/sysdeps/unix/sysv/linux/Makefile
> +++ b/sysdeps/unix/sysv/linux/Makefile
> @@ -107,7 +107,7 @@ tests += tst-clone tst-clone2 tst-clone3 tst-fanotify tst-personality \
>  	 test-errno-linux tst-memfd_create tst-mlock2 tst-pkey \
>  	 tst-rlimit-infinity tst-ofdlocks tst-gettid tst-gettid-kill \
>  	 tst-tgkill tst-sysvsem-linux tst-sysvmsg-linux tst-sysvshm-linux \
> -	 tst-timerfd tst-ppoll
> +	 tst-timerfd tst-ppoll tst-futimens
>  tests-internal += tst-ofdlocks-compat tst-sigcontext-get_pc
>  
>  CFLAGS-tst-sigcontext-get_pc.c = -fasynchronous-unwind-tables
> diff --git a/sysdeps/unix/sysv/linux/tst-futimens.c b/sysdeps/unix/sysv/linux/tst-futimens.c
> new file mode 100644
> index 0000000000..ddc191b0b2
> --- /dev/null
> +++ b/sysdeps/unix/sysv/linux/tst-futimens.c
> @@ -0,0 +1,80 @@
> +/* Test for futimens
> +   Copyright (C) 2021 Free Software Foundation, Inc.
> +   This file is part of the GNU C Library.
> +
> +   The GNU C Library is free software; you can redistribute it and/or
> +   modify it under the terms of the GNU Lesser General Public
> +   License as published by the Free Software Foundation; either
> +   version 2.1 of the License, or (at your option) any later version.
> +
> +   The GNU C Library is distributed in the hope that it will be useful,
> +   but WITHOUT ANY WARRANTY; without even the implied warranty of
> +   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> +   Lesser General Public License for more details.
> +
> +   You should have received a copy of the GNU Lesser General Public
> +   License along with the GNU C Library; if not, see
> +   <https://www.gnu.org/licenses/>.  */
> +
> +#include <time.h>
> +#include <stdlib.h>
> +#include <sys/stat.h>
> +#include <support/check.h>
> +#include <support/xunistd.h>
> +#include <support/temp_file.h>
> +
> +static int temp_fd = -1;
> +
> +/* struct timespec array with Y2038 threshold minus 2 and 1 seconds.  */
> +const struct timespec t1[2] = { { 0x7FFFFFFE, 0 },  { 0x7FFFFFFF, 0 } };
> +
> +/* struct timespec array with Y2038 threshold plus 1 and 2 seconds.  */
> +const struct timespec t2[2] = { { 0x80000001ULL, 0 },  { 0x80000002ULL, 0 } };
> +
> +/* struct timespec array around Y2038 threshold.  */
> +const struct timespec t3[2] = { { 0x7FFFFFFE, 0 },  { 0x80000002ULL, 0 } };
> +
> +#define PREPARE do_prepare
> +static void
> +do_prepare (int argc, char *argv[])
> +{
> +  temp_fd = create_temp_file ("futimensat", NULL);
> +  TEST_VERIFY_EXIT (temp_fd > 0);
> +}
> +
> +static int
> +test_futimens_helper (const struct timespec *ts)
> +{
> +  struct stat64 st;
> +  int result;
> +  time_t t;
> +
> +  /* Check if we run on port with 32 bit time_t size */
> +  if (__builtin_add_overflow (ts->tv_sec, 0, &t))
> +    return 0;
> +
> +  result = futimens(temp_fd, ts);

Space before '('.

> +  TEST_VERIFY_EXIT (result == 0);
> +
> +  xfstat (temp_fd, &st);
> +
> +  /* Check if seconds for atime match */
> +  TEST_COMPARE (st.st_atime, ts[0].tv_sec);
> +
> +  /* Check if seconds for mtime match */
> +  TEST_COMPARE (st.st_mtime, ts[1].tv_sec);
> +
> +  return 0;
> +}
> +
> +static int
> +do_test (void)
> +{
> +  test_futimens_helper (&t1[0]);
> +  test_futimens_helper (&t2[0]);
> +  test_futimens_helper (&t3[0]);
> +
> +  return 0;
> +}
> +
> +#include <support/test-driver.c>
> 

Ok.
diff mbox series

Patch

diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile
index eca24cdfe8..4e043c78ab 100644
--- a/sysdeps/unix/sysv/linux/Makefile
+++ b/sysdeps/unix/sysv/linux/Makefile
@@ -107,7 +107,7 @@  tests += tst-clone tst-clone2 tst-clone3 tst-fanotify tst-personality \
 	 test-errno-linux tst-memfd_create tst-mlock2 tst-pkey \
 	 tst-rlimit-infinity tst-ofdlocks tst-gettid tst-gettid-kill \
 	 tst-tgkill tst-sysvsem-linux tst-sysvmsg-linux tst-sysvshm-linux \
-	 tst-timerfd tst-ppoll
+	 tst-timerfd tst-ppoll tst-futimens
 tests-internal += tst-ofdlocks-compat tst-sigcontext-get_pc
 
 CFLAGS-tst-sigcontext-get_pc.c = -fasynchronous-unwind-tables
diff --git a/sysdeps/unix/sysv/linux/tst-futimens.c b/sysdeps/unix/sysv/linux/tst-futimens.c
new file mode 100644
index 0000000000..ddc191b0b2
--- /dev/null
+++ b/sysdeps/unix/sysv/linux/tst-futimens.c
@@ -0,0 +1,80 @@ 
+/* Test for futimens
+   Copyright (C) 2021 Free Software Foundation, Inc.
+   This file is part of the GNU C Library.
+
+   The GNU C Library is free software; you can redistribute it and/or
+   modify it under the terms of the GNU Lesser General Public
+   License as published by the Free Software Foundation; either
+   version 2.1 of the License, or (at your option) any later version.
+
+   The GNU C Library is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   Lesser General Public License for more details.
+
+   You should have received a copy of the GNU Lesser General Public
+   License along with the GNU C Library; if not, see
+   <https://www.gnu.org/licenses/>.  */
+
+#include <time.h>
+#include <stdlib.h>
+#include <sys/stat.h>
+#include <support/check.h>
+#include <support/xunistd.h>
+#include <support/temp_file.h>
+
+static int temp_fd = -1;
+
+/* struct timespec array with Y2038 threshold minus 2 and 1 seconds.  */
+const struct timespec t1[2] = { { 0x7FFFFFFE, 0 },  { 0x7FFFFFFF, 0 } };
+
+/* struct timespec array with Y2038 threshold plus 1 and 2 seconds.  */
+const struct timespec t2[2] = { { 0x80000001ULL, 0 },  { 0x80000002ULL, 0 } };
+
+/* struct timespec array around Y2038 threshold.  */
+const struct timespec t3[2] = { { 0x7FFFFFFE, 0 },  { 0x80000002ULL, 0 } };
+
+#define PREPARE do_prepare
+static void
+do_prepare (int argc, char *argv[])
+{
+  temp_fd = create_temp_file ("futimensat", NULL);
+  TEST_VERIFY_EXIT (temp_fd > 0);
+}
+
+static int
+test_futimens_helper (const struct timespec *ts)
+{
+  struct stat64 st;
+  int result;
+  time_t t;
+
+  /* Check if we run on port with 32 bit time_t size */
+  if (__builtin_add_overflow (ts->tv_sec, 0, &t))
+    return 0;
+
+  result = futimens(temp_fd, ts);
+  TEST_VERIFY_EXIT (result == 0);
+
+  xfstat (temp_fd, &st);
+
+  /* Check if seconds for atime match */
+  TEST_COMPARE (st.st_atime, ts[0].tv_sec);
+
+  /* Check if seconds for mtime match */
+  TEST_COMPARE (st.st_mtime, ts[1].tv_sec);
+
+  return 0;
+}
+
+static int
+do_test (void)
+{
+  test_futimens_helper (&t1[0]);
+  test_futimens_helper (&t2[0]);
+  test_futimens_helper (&t3[0]);
+
+  return 0;
+}
+
+#include <support/test-driver.c>