From patchwork Fri Jun 30 20:45:34 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sunil Pandey X-Patchwork-Id: 1802145 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@legolas.ozlabs.org Authentication-Results: legolas.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=sourceware.org (client-ip=2620:52:3:1:0:246e:9693:128c; helo=sourceware.org; envelope-from=libc-alpha-bounces+incoming=patchwork.ozlabs.org@sourceware.org; receiver=) Authentication-Results: legolas.ozlabs.org; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=ce2exe6q; dkim-atps=neutral Received: from sourceware.org (server2.sourceware.org [IPv6:2620:52:3:1:0:246e:9693:128c]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384) server-digest SHA384) (No client certificate requested) by legolas.ozlabs.org (Postfix) with ESMTPS id 4Qt6mW69Jtz20ZV for ; Sat, 1 Jul 2023 06:46:19 +1000 (AEST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 0A17C3882AD1 for ; Fri, 30 Jun 2023 20:46:17 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0A17C3882AD1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1688157977; bh=MqKHX57yEflXRm/Ym2S7FKPp+/i4oJo7pYDXjlF7up4=; h=To:Cc:Subject:Date:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From:Reply-To:From; b=ce2exe6qjJduJEd4Hv+6cxkVv9k32LAUztBtpUVkyMQyESB+/sAsXtKeD6GcxVj4Y qtgMwRtmnX6mAl1pjRKTkp+riwEsQsm0XgaLL0VSrojA/rUzUv+HCbdw3HUDUchP0i XCYDN1w+mjIsjFvZf/j3yQN8lBC6ErnSPvm+EE6c= X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by sourceware.org (Postfix) with ESMTPS id 20A163882AE2 for ; Fri, 30 Jun 2023 20:45:37 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 20A163882AE2 X-IronPort-AV: E=McAfee;i="6600,9927,10757"; a="362530756" X-IronPort-AV: E=Sophos;i="6.01,171,1684825200"; d="scan'208";a="362530756" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Jun 2023 13:45:35 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10757"; a="721092653" X-IronPort-AV: E=Sophos;i="6.01,171,1684825200"; d="scan'208";a="721092653" Received: from scymds04.sc.intel.com ([10.82.73.238]) by fmsmga007.fm.intel.com with ESMTP; 30 Jun 2023 13:45:35 -0700 Received: from gskx-1.sc.intel.com (gskx-1.sc.intel.com [172.25.149.211]) by scymds04.sc.intel.com (Postfix) with ESMTP id 77BB01826763; Fri, 30 Jun 2023 13:45:35 -0700 (PDT) To: libc-alpha@sourceware.org Cc: hjl.tools@gmail.com Subject: [PATCH] Add strlcpy/wcslcpy testcase Date: Fri, 30 Jun 2023 13:45:34 -0700 Message-Id: <20230630204534.2059804-1-skpgkp2@gmail.com> X-Mailer: git-send-email 2.38.1 MIME-Version: 1.0 X-Spam-Status: No, score=-8.7 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, NML_ADSP_CUSTOM_MED, SPF_HELO_NONE, SPF_SOFTFAIL, SPOOFED_FREEMAIL, SPOOF_GMAIL_MID, 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+incoming=patchwork.ozlabs.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. --- string/Makefile | 1 + string/test-strlcpy.c | 325 ++++++++++++++++++++++++++++++++++++++++++ wcsmbs/Makefile | 1 + wcsmbs/test-wcslcpy.c | 20 +++ 4 files changed, 347 insertions(+) create mode 100644 string/test-strlcpy.c create mode 100644 wcsmbs/test-wcslcpy.c diff --git a/string/Makefile b/string/Makefile index d3106d10a9..2bae2ba194 100644 --- a/string/Makefile +++ b/string/Makefile @@ -160,6 +160,7 @@ tests := \ test-strcpy \ test-strcspn \ test-strdup \ + test-strlcpy \ test-strlen \ test-strncasecmp \ test-strncat \ diff --git a/string/test-strlcpy.c b/string/test-strlcpy.c new file mode 100644 index 0000000000..43a721760f --- /dev/null +++ b/string/test-strlcpy.c @@ -0,0 +1,325 @@ +/* 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 + . */ + +#ifdef WIDE +# include +# define CHAR wchar_t +# define BIG_CHAR WCHAR_MAX +# define SMALL_CHAR 1273 +# define MEMCMP wmemcmp +# define MEMSET wmemset +# define STRLEN wcslen +#else +# define CHAR char +# define BIG_CHAR CHAR_MAX +# define SMALL_CHAR 127 +# define MEMCMP memcmp +# define MEMSET memset +# define STRLEN strlen +#endif /* !WIDE */ + + +#ifndef SIMPLE_STRLCPY +# define TEST_MAIN +# ifndef WIDE +# define TEST_NAME "strlcpy" +# else +# define TEST_NAME "wcslcpy" +# endif /* WIDE */ +# include "test-string.h" +# ifndef WIDE +# define SIMPLE_STRLCPY simple_strlcpy +# define STRLCPY strlcpy +# else +# define SIMPLE_STRLCPY simple_wcslcpy +# define STRLCPY wcslcpy +# endif /* WIDE */ + + +IMPL (STRLCPY, 1) + +/* Naive implementation to verify results. */ +size_t +SIMPLE_STRLCPY (CHAR *dst, const CHAR *src, size_t n) +{ + size_t ret = STRLEN (src); + while (n--) + if ((*dst++ = *src++) == '\0') + return ret; + *dst = '\0'; + return ret; +} + +#endif /* !SIMPLE_STRLCPY */ + +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 22192985e1..3fde8c76db 100644 --- a/wcsmbs/Makefile +++ b/wcsmbs/Makefile @@ -134,6 +134,7 @@ tests := \ test-wcscpy \ test-wcscspn \ test-wcsdup \ + test-wcslcpy \ test-wcslen \ test-wcsncat \ test-wcsncmp \ diff --git a/wcsmbs/test-wcslcpy.c b/wcsmbs/test-wcslcpy.c new file mode 100644 index 0000000000..91af5ae511 --- /dev/null +++ b/wcsmbs/test-wcslcpy.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/test-strlcpy.c"