From patchwork Wed Jan 27 12:47:10 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lukasz Majewski X-Patchwork-Id: 41833 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id B9AC4396E841; Wed, 27 Jan 2021 12:47:30 +0000 (GMT) X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from mail-out.m-online.net (mail-out.m-online.net [IPv6:2001:a60:0:28:0:1:25:1]) by sourceware.org (Postfix) with ESMTPS id A0509395B818 for ; Wed, 27 Jan 2021 12:47:26 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org A0509395B818 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=denx.de Authentication-Results: sourceware.org; spf=none smtp.mailfrom=lukma@denx.de Received: from frontend01.mail.m-online.net (unknown [192.168.8.182]) by mail-out.m-online.net (Postfix) with ESMTP id 4DQjzx332Hz1ryp6; Wed, 27 Jan 2021 13:47:25 +0100 (CET) Received: from localhost (dynscan1.mnet-online.de [192.168.6.70]) by mail.m-online.net (Postfix) with ESMTP id 4DQjzx1YTlz1sP6h; Wed, 27 Jan 2021 13:47:25 +0100 (CET) X-Virus-Scanned: amavisd-new at mnet-online.de Received: from mail.mnet-online.de ([192.168.8.182]) by localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new, port 10024) with ESMTP id EyJyr80WqhHT; Wed, 27 Jan 2021 13:47:22 +0100 (CET) X-Auth-Info: Z8mXwm+b0LZhLhZ0pJCw6V86JI2Qpu+PNoBoBZ6iMEo= Received: from localhost.localdomain (89-64-25-12.dynamic.chello.pl [89.64.25.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.mnet-online.de (Postfix) with ESMTPSA; Wed, 27 Jan 2021 13:47:22 +0100 (CET) From: Lukasz Majewski To: Joseph Myers , Adhemerval Zanella , Florian Weimer Subject: [PATCH v2] tst: Provide Y2038 tests for mktime (tst-mktime4.c) Date: Wed, 27 Jan 2021 13:47:10 +0100 Message-Id: <20210127124710.21057-1-lukma@denx.de> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 X-Spam-Status: No, score=-9.3 required=5.0 tests=BAYES_00, GIT_PATCH_0, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, KAM_SHORT, RCVD_IN_ABUSEAT, RCVD_IN_BARRACUDACENTRAL, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: GNU C Library , Florian Weimer , Alistair Francis Errors-To: libc-alpha-bounces@sourceware.org Sender: "Libc-alpha" This change adds new test to assess mktime's functionality. To be more specific - following use cases are checked: - Pass struct tm as epoch time - Pass struct tm as value just before Y2038 threshold (returned value shall be 0x7FFFFFFF) - Pass struct tm as the first value after Y2038 threshold (expected value - 0x80000000) --- Changes for v2: - Remove FAIL_UNSUPPORTED() when sizeof (time_t) <= 4 - Remove tzset () as it is already done in mktime () - Use TEST_COMPARE to check if correct result is received - Use "TZ=:" glibc extension to guarantee UTC time zone - Add two more checks - prepare struct tm in a way that mktime will return 0xFFFFFFFF and 0x100000000 --- time/Makefile | 4 +- time/tst-mktime4.c | 100 +++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 103 insertions(+), 1 deletion(-) create mode 100644 time/tst-mktime4.c diff --git a/time/Makefile b/time/Makefile index 7de2ce0196..1c8678e46b 100644 --- a/time/Makefile +++ b/time/Makefile @@ -51,7 +51,7 @@ tests := test_time clocktest tst-posixtz tst-strptime tst_wcsftime \ tst-clock tst-clock2 tst-clock_nanosleep tst-cpuclock1 \ tst-adjtime tst-clock-y2038 tst-clock2-y2038 \ tst-cpuclock1-y2038 tst-clock_nanosleep-y2038 tst-clock_settime \ - tst-clock_adjtime tst-ctime tst-difftime + tst-clock_adjtime tst-ctime tst-difftime tst-mktime4 include ../Rules @@ -88,4 +88,6 @@ test_time-ARGS= EST5EDT CST tst-tzname-ENV = TZDIR=${common-objpfx}timezone/testdata CPPFLAGS-tst-tzname.c += -DTZDEFRULES='"$(posixrules-file)"' +CFLAGS-tst-mktime4.c += -Wno-overflow + bug-getdate1-ARGS = ${objpfx}bug-getdate1-fmt diff --git a/time/tst-mktime4.c b/time/tst-mktime4.c new file mode 100644 index 0000000000..811120721e --- /dev/null +++ b/time/tst-mktime4.c @@ -0,0 +1,100 @@ +/* Test for mktime (4) + 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 + . */ + +#include +#include +#include + +struct tm tm0 = { + .tm_year = 70, + .tm_mon = 0, + .tm_mday = 1, + .tm_hour = 0, + .tm_min = 0, + .tm_sec = 0, + .tm_wday = 4, + .tm_yday = 0 +}; + +struct tm tmY2038 = { + .tm_year = 138, + .tm_mon = 0, + .tm_mday = 19, + .tm_hour = 3, + .tm_min = 14, + .tm_sec = 7, + .tm_wday = 2, + .tm_yday = 18 +}; + +struct tm tm1 = { + .tm_year = 206, + .tm_mon = 1, + .tm_mday = 7, + .tm_hour = 6, + .tm_min = 28, + .tm_sec = 15, + .tm_wday = 0, + .tm_yday = 0 +}; + +static +void test_mktime_helper (struct tm *tm, time_t exp_val) +{ + time_t result = mktime (tm); + if (result == -1) + FAIL_EXIT1 ("*** mktime failed: %m"); + + TEST_COMPARE (result, exp_val); +} + +static int +do_test (void) +{ + /* Use glibc time zone extension "TZ=:" to to guarantee that UTC + without leap seconds is used for the test. */ + TEST_VERIFY_EXIT (setenv ("TZ", ":", 1) == 0); + + /* Check that mktime(1970-01-01 00:00:00) returns 0. */ + test_mktime_helper (&tm0, 0); + + /* Check that mktime(2038-01-19 03:14:07) returns 0x7FFFFFFF. */ + test_mktime_helper (&tmY2038, 0x7fffffff); + + if (sizeof (time_t) > 4) + { + /* Check that mktime(2038-01-19 03:14:08) returns 0x80000000 + (time_t overflow). */ + tmY2038.tm_sec++; + test_mktime_helper (&tmY2038, 0x80000000); + + /* Check that mktime(2106-02-07 06:28:15) returns 0xFFFFFFFF. */ + test_mktime_helper (&tm1, 0xFFFFFFFF); + + /* Check that mktime(2106-02-07 06:28:16) returns 0x100000000. + + This test is executed with -Wno-overflow to prevent build + breaks when it is compiled on ports with 32 bit time_t. */ + tm1.tm_sec++; + test_mktime_helper (&tm1, 0x100000000); + } + + return 0; +} + +#include