From patchwork Thu Jul 27 17:51:16 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sunil Pandey X-Patchwork-Id: 73280 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 9A7E4385AFA9 for ; Thu, 27 Jul 2023 18:02:44 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 9A7E4385AFA9 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1690480964; bh=t785kczGh+9hUTrPuwtnn+ChTJEs6LFMq0fFvYu3yDc=; h=To:Cc:Subject:Date:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=EKSVdvd8ZtvBB2iPKVozdteRx462lG2krFAvf+fWeN5KgAfxC2EHzHd7JCmwoNWeJ G6ycgoO9ldFH1iq4IDIcRooej7QpNV7R7dMnE8HeXGOA7HKLVyn3x2AxTa5uTHCyjW ypWGlL5fb2O1tYtJ/r4PKrcIiGIcSEFTXyegWI2k= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from mgamail.intel.com (unknown [134.134.136.24]) by sourceware.org (Postfix) with ESMTPS id A23193856964 for ; Thu, 27 Jul 2023 18:02:16 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A23193856964 X-IronPort-AV: E=McAfee;i="6600,9927,10784"; a="371088257" X-IronPort-AV: E=Sophos;i="6.01,235,1684825200"; d="scan'208";a="371088257" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Jul 2023 11:00:44 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10784"; a="762301092" X-IronPort-AV: E=Sophos;i="6.01,235,1684825200"; d="scan'208";a="762301092" Received: from scymds03.sc.intel.com ([10.148.94.166]) by orsmga001.jf.intel.com with ESMTP; 27 Jul 2023 10:51:17 -0700 Received: from gskx-1.sc.intel.com (gskx-1.sc.intel.com [172.25.149.211]) by scymds03.sc.intel.com (Postfix) with ESMTP id E98F664; Thu, 27 Jul 2023 10:51:16 -0700 (PDT) To: libc-alpha@sourceware.org Cc: hjl.tools@gmail.com Subject: [PATCH v2] Add strlcpy/wcslcpy testcase Date: Thu, 27 Jul 2023 10:51:16 -0700 Message-ID: <20230727175116.2793579-1-skpgkp2@gmail.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: References: MIME-Version: 1.0 X-Spam-Status: No, score=-8.8 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, FORGED_GMAIL_RCVD, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, GIT_PATCH_0, HK_RANDOM_ENVFROM, HK_RANDOM_FROM, KAM_DMARC_NONE, KAM_DMARC_STATUS, KAM_SHORT, MAY_BE_FORGED, NML_ADSP_CUSTOM_MED, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_SOFTFAIL, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) 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: , X-Patchwork-Original-From: Sunil K Pandey via Libc-alpha From: Sunil Pandey Reply-To: Sunil K Pandey Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org Sender: "Libc-alpha" This patch implements comprehensive tests for strlcpy/wcslcpy functions. Tests are mostly derived from strncpy test suites and modified to incorporate strlcpy/wcslcpy specifications. Changes from v1: Rename test-strlcpy to tst-strlcpy2. Rename test-wcslcpy to tst-wcslcpy2. Remove unused SIMPLE_STRLCPY function. Rearrange defines in alphabetical order. --- string/Makefile | 1 + string/tst-strlcpy2.c | 303 ++++++++++++++++++++++++++++++++++++++++++ wcsmbs/Makefile | 1 + wcsmbs/tst-wcslcpy2.c | 20 +++ 4 files changed, 325 insertions(+) create mode 100644 string/tst-strlcpy2.c create mode 100644 wcsmbs/tst-wcslcpy2.c diff --git a/string/Makefile b/string/Makefile index 8cdfd5b000..cecaa796a8 100644 --- a/string/Makefile +++ b/string/Makefile @@ -199,6 +199,7 @@ tests := \ tst-strfry \ tst-strlcat \ tst-strlcpy \ + tst-strlcpy2 \ tst-strlen \ tst-strtok \ tst-strtok_r \ diff --git a/string/tst-strlcpy2.c b/string/tst-strlcpy2.c new file mode 100644 index 0000000000..4d6ebb0ebd --- /dev/null +++ b/string/tst-strlcpy2.c @@ -0,0 +1,303 @@ +/* Test strlcpy functions. + Copyright (C) 2023 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 + . */ + +#define TEST_MAIN +#ifndef WIDE +# define TEST_NAME "strlcpy" +#else +# define TEST_NAME "wcslcpy" +#endif /* WIDE */ +#include "test-string.h" + +#ifdef WIDE +# include +# define BIG_CHAR WCHAR_MAX +# define CHAR wchar_t +# define MEMCMP wmemcmp +# define MEMSET wmemset +# define SMALL_CHAR 1273 +# define STRLCPY wcslcpy +# define STRLEN wcslen +#else +# define BIG_CHAR CHAR_MAX +# define CHAR char +# define MEMCMP memcmp +# define MEMSET memset +# define SMALL_CHAR 127 +# define STRLCPY strlcpy +# define STRLEN strlen +#endif /* !WIDE */ + +IMPL (STRLCPY, 1) + +typedef size_t (*proto_t) (CHAR *, const CHAR *, size_t); + +static void +do_one_test (impl_t *impl, CHAR *dst, const CHAR *src, size_t len, size_t n) +{ + if (CALL (impl, dst, src, n) != len) + { + error (0, 0, "Wrong result in function %s %zd %zd", impl->name, + CALL (impl, dst, src, n), len); + ret = 1; + return; + } + + if (n == 0) + return; + + len = (len >= n ? n - 1 : len); + if (MEMCMP (dst, src, len) != 0) + { + error (0, 0, "Wrong result in function1 %s", impl->name); + ret = 1; + return; + } + + if (dst [len] != '\0') + { + error (0, 0, "Wrong result in function2 %s", impl->name); + ret = 1; + return; + } +} + +static void +do_test (size_t align1, size_t align2, size_t len, size_t n, int max_char) +{ + size_t i; + CHAR *s1, *s2; + + /* For wcslcpy: align1 and align2 here mean alignment not in bytes, + but in wchar_ts, in bytes it will equal to align * (sizeof (wchar_t)). */ + align1 &= 7; + if ((align1 + len) * sizeof (CHAR) >= page_size) + return; + + align2 &= 7; + if ((align2 + len) * sizeof (CHAR) >= page_size) + return; + + s1 = (CHAR *) (buf1) + align1; + s2 = (CHAR *) (buf2) + align2; + + for (i = 0; i < len; ++i) + s1[i] = 32 + 23 * i % (max_char - 32); + s1[len] = 0; + + FOR_EACH_IMPL (impl, 0) + do_one_test (impl, s2, s1, len, n); +} + +static void +do_page_tests (void) +{ + CHAR *s1, *s2; + const size_t maxoffset = 64; + + /* Put s1 at the maxoffset from the edge of buf1's last page. */ + s1 = (CHAR *) buf1 + BUF1PAGES * page_size / sizeof(CHAR) - maxoffset; + /* s2 needs room to put a string with size of maxoffset + 1 at s2 + + (maxoffset - 1). */ + s2 = (CHAR *) buf2 + page_size / sizeof(CHAR) - maxoffset * 2; + + MEMSET (s1, 'a', maxoffset - 1); + s1[maxoffset - 1] = '\0'; + + /* Both strings are bounded to a page with read/write access and the next + page is protected with PROT_NONE (meaning that any access outside of the + page regions will trigger an invalid memory access). + + The loop copies the string s1 for all possible offsets up to maxoffset + for both inputs with a size larger than s1 (so memory access outside the + expected memory regions might trigger invalid access). */ + + for (size_t off1 = 0; off1 < maxoffset; off1++) + { + for (size_t off2 = 0; off2 < maxoffset; off2++) + { + FOR_EACH_IMPL (impl, 0) + do_one_test (impl, s2 + off2, s1 + off1, maxoffset - off1 - 1, + maxoffset + (maxoffset - off2)); + } + } +} + +static void +do_random_tests (void) +{ + size_t i, j, n, align1, align2, len, size, mode; + CHAR *p1 = (CHAR *) (buf1 + page_size) - 1024; + CHAR *p2 = (CHAR *) (buf2 + page_size) - 1024; + size_t res; + + for (n = 0; n < ITERATIONS; n++) + { + /* For wcslcpy: align1 and align2 here mean align not in bytes, + but in wchar_ts, in bytes it will equal to align * (sizeof + (wchar_t)). */ + + mode = random (); + if (mode & 1) + { + size = random () & 255; + align1 = 512 - size - (random () & 15); + if (mode & 2) + align2 = align1 - (random () & 24); + else + align2 = align1 - (random () & 31); + if (mode & 4) + { + j = align1; + align1 = align2; + align2 = j; + } + if (mode & 8) + len = size - (random () & 31); + else + len = 512; + if (len >= 512) + len = random () & 511; + } + else + { + align1 = random () & 31; + if (mode & 2) + align2 = random () & 31; + else + align2 = align1 + (random () & 24); + len = random () & 511; + j = align1; + if (align2 > j) + j = align2; + if (mode & 4) + { + size = random () & 511; + if (size + j > 512) + size = 512 - j - (random () & 31); + } + else + size = 512 - j; + if ((mode & 8) && len + j >= 512) + len = 512 - j - (random () & 7); + } + j = len + align1; + for (i = 0; i < j; i++) + { + p1[i] = random () & BIG_CHAR; + if (i >= align1 && i < len + align1 && !p1[i]) + p1[i] = (random () & SMALL_CHAR) + 3; + } + p1[i] = 0; + + FOR_EACH_IMPL (impl, 1) + { + MEMSET (p2 - 64, '\1', 512 + 64); + res = CALL (impl, (CHAR *) (p2 + align2), + (CHAR *) (p1 + align1), size); + if (res != len) + { + error (0, 0, "Iteration %zd - wrong result in function %s (%zd, %zd) %zd != %zd", + n, impl->name, align1, align2, len, res); + ret = 1; + } + for (j = 0; j < align2 + 64; ++j) + { + if (p2[j - 64] != '\1') + { + error (0, 0, "Iteration %zd - garbage before, %s (%zd, %zd, %zd)", + n, impl->name, align1, align2, len); + ret = 1; + break; + } + } + j = align2 + len + 1; + if (size + align2 > j) + j = size + align2; + for (; j < 512; ++j) + { + if (p2[j] != '\1') + { + error (0, 0, "Iteration %zd - garbage after, %s (%zd, %zd, %zd)", + n, impl->name, align1, align2, len); + ret = 1; + break; + } + } + j = len; + /* Check for zero size. */ + if (size) + { + if (size <= j) + j = size - 1; + if (MEMCMP (p1 + align1, p2 + align2, j)) + { + error (0, 0, "Iteration %zd - different strings, %s (%zd, %zd, %zd)", + n, impl->name, align1, align2, len); + ret = 1; + } + if (p2[align2 + j]) + { + error (0, 0, "Iteration %zd - garbage after size, %s (%zd, %zd, %zd)", + n, impl->name, align1, align2, len); + ret = 1; + break; + } + } + } + } +} + +int +test_main (void) +{ + size_t i; + + test_init (); + + printf ("%28s", ""); + FOR_EACH_IMPL (impl, 0) + printf ("\t%s", impl->name); + putchar ('\n'); + + for (i = 1; i < 8; ++i) + { + do_test (i, i, 16, 16, SMALL_CHAR); + do_test (i, i, 16, 16, BIG_CHAR); + do_test (i, 2 * i, 16, 16, SMALL_CHAR); + do_test (2 * i, i, 16, 16, BIG_CHAR); + do_test (8 - i, 2 * i, 1 << i, 2 << i, SMALL_CHAR); + do_test (2 * i, 8 - i, 2 << i, 1 << i, SMALL_CHAR); + do_test (8 - i, 2 * i, 1 << i, 2 << i, BIG_CHAR); + do_test (2 * i, 8 - i, 2 << i, 1 << i, BIG_CHAR); + } + + for (i = 1; i < 8; ++i) + { + do_test (0, 0, 4 << i, 8 << i, SMALL_CHAR); + do_test (0, 0, 16 << i, 8 << i, SMALL_CHAR); + do_test (8 - i, 2 * i, 4 << i, 8 << i, SMALL_CHAR); + do_test (8 - i, 2 * i, 16 << i, 8 << i, SMALL_CHAR); + } + + do_random_tests (); + do_page_tests (); + return ret; +} + +#include diff --git a/wcsmbs/Makefile b/wcsmbs/Makefile index 431136b9c9..d7626b386d 100644 --- a/wcsmbs/Makefile +++ b/wcsmbs/Makefile @@ -180,6 +180,7 @@ tests := \ tst-wcrtomb \ tst-wcslcat \ tst-wcslcpy \ + tst-wcslcpy2 \ tst-wcsnlen \ tst-wcstod-nan-locale \ tst-wcstod-nan-sign \ diff --git a/wcsmbs/tst-wcslcpy2.c b/wcsmbs/tst-wcslcpy2.c new file mode 100644 index 0000000000..bec40ed135 --- /dev/null +++ b/wcsmbs/tst-wcslcpy2.c @@ -0,0 +1,20 @@ +/* Test wcslcpy functions. + Copyright (C) 2023 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 + . */ + +#define WIDE 1 +#include "../string/tst-strlcpy2.c"