From nobody Sat May 30 17:43:40 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=linux.alibaba.com ARC-Seal: i=1; a=rsa-sha256; t=1780122568; cv=none; d=zohomail.com; s=zohoarc; b=H36QO0qGB2b0ZOeUGIPsg0DIBpTW6e79iIPQ5AXWG2Sr99h4bySLoP2l7BTQaSMqbSKDB9RJ4xGcZRAMwwdHsrMEPwhwYkuokQB9YCb+bTJ1Vs0DDaKFJZXNC9PESlW/qt6k6KLRDMXDMyaLEU/CUuv6NRCWh9l5M2rTIRmrGQs= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1780122568; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=z4y8uU2x8AV1KIYxLjQG+6/0XGc9ZwUDlfvJk+6eJYo=; b=e4f2A33kwJISA7VgrWmdE5vT4AqoDmHfTjO//maAg8j3l1MQmil1HAt69R2SwzwyJu3tlQQnFhxozK3xfxF+rOwEPU+i0//A99JS3WQb5eRbnhE1lUzL1UNMJU8HFq8ezhj1rpnR3yFhrZBTkITNMVtvRkQla178WJPLk2Kmguk= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists1p.gnu.org (lists1p.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1780122568115457.56352076457165; Fri, 29 May 2026 23:29:28 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists1p.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1wTDBk-00074v-1N; Sat, 30 May 2026 02:28:52 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists1p.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1wTDBi-00074n-2Z for qemu-devel@nongnu.org; Sat, 30 May 2026 02:28:50 -0400 Received: from [115.124.30.111] (helo=out30-111.freemail.mail.aliyun.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1wTDBc-0005qc-6b for qemu-devel@nongnu.org; Sat, 30 May 2026 02:28:49 -0400 Received: from localhost(mailfrom:guobin@linux.alibaba.com fp:SMTPD_---0X3r4Fqm_1780122497 cluster:ay36) by smtp.aliyun-inc.com; Sat, 30 May 2026 14:28:18 +0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.alibaba.com; s=default; t=1780122499; h=From:To:Subject:Date:Message-ID:MIME-Version:Content-Type; bh=z4y8uU2x8AV1KIYxLjQG+6/0XGc9ZwUDlfvJk+6eJYo=; b=gCe/7mnGV/slPUAx7Rv4nCiT5PYmeMFg7aV+acPVU1SMUaiJyx6toao8tIHvpl760YCBC++LQUe8Ii7ku7HEv/lg5x0TDkwx0vfrkweYVCBPRvsHF4GcNoxL53tfP4t6gu4mqjFbdvj8V2bn7cTsTq567SMJVGGz5CepKL+IeeQ= X-Alimail-AntiSpam: AC=PASS; BC=-1|-1; BR=01201311R161e4; CH=green; DM=||false|; DS=||; FP=0|-1|-1|-1|0|-1|-1|-1; HT=maildocker-contentspam011083073210; MF=guobin@linux.alibaba.com; NM=1; PH=DS; RN=9; SR=0; TI=SMTPD_---0X3r4Fqm_1780122497; From: Bin Guo To: qemu-devel@nongnu.org Cc: Warner Losh , Kyle Evans , Laurent Vivier , =?UTF-8?q?Philippe=20Mathieu-Daud=C3=A9?= , =?UTF-8?q?Alex=20Benn=C3=A9e?= , =?UTF-8?q?Daniel=20P=20=2E=20Berrang=C3=A9?= , Thomas Huth , Markus Armbruster Subject: [PATCH v2] util/cutils: drop qemu_strnlen() in favor of strnlen() Date: Sat, 30 May 2026 14:28:16 +0800 Message-ID: <20260530062816.59206-1-guobin@linux.alibaba.com> X-Mailer: git-send-email 2.50.1 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-Host-Lookup-Failed: Reverse DNS lookup failed for 115.124.30.111 (deferred) Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists1p.gnu.org; Received-SPF: pass client-ip=115.124.30.111; envelope-from=guobin@linux.alibaba.com; helo=out30-111.freemail.mail.aliyun.com X-Spam_score_int: -166 X-Spam_score: -16.7 X-Spam_bar: ---------------- X-Spam_report: (-16.7 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, ENV_AND_HDR_SPF_MATCH=-0.5, RCVD_IN_DNSWL_NONE=-0.0001, RDNS_NONE=0.793, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, UNPARSEABLE_RELAY=0.001, USER_IN_DEF_DKIM_WL=-7.5, USER_IN_DEF_SPF_WL=-7.5 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: qemu development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @linux.alibaba.com) X-ZM-MESSAGEID: 1780122570038154100 There are only three call sites, and strnlen() is available on all supported platforms (POSIX.1-2008, Windows via UCRT, MinGW). Remove the hand-rolled wrapper and use the standard function directly. While here, align bsd-user/uaccess.c to use size_t for max_len/len, matching linux-user/uaccess.c and eliminating a signed/unsigned mismatch. Also remove the stale qemu_strnlen() entry from docs/devel/style.rst. Suggested-by: Philippe Mathieu-Daud=C3=A9 Reviewed-by: Philippe Mathieu-Daud=C3=A9 Signed-off-by: Bin Guo Reviewed-by: Alex Benn=C3=A9e Reviewed-by: Warner Losh --- bsd-user/uaccess.c | 4 ++-- docs/devel/style.rst | 1 - include/qemu/cutils.h | 17 +---------------- linux-user/uaccess.c | 2 +- util/cutils.c | 15 +-------------- 5 files changed, 5 insertions(+), 34 deletions(-) diff --git a/bsd-user/uaccess.c b/bsd-user/uaccess.c index 89163257f4..7ad4b580f4 100644 --- a/bsd-user/uaccess.c +++ b/bsd-user/uaccess.c @@ -43,7 +43,7 @@ abi_long target_strlen(abi_ulong guest_addr1) { uint8_t *ptr; abi_ulong guest_addr; - int max_len, len; + size_t max_len, len; =20 guest_addr =3D guest_addr1; for (;;) { @@ -51,7 +51,7 @@ abi_long target_strlen(abi_ulong guest_addr1) ptr =3D lock_user(VERIFY_READ, guest_addr, max_len, 1); if (!ptr) return -TARGET_EFAULT; - len =3D qemu_strnlen((const char *)ptr, max_len); + len =3D strnlen((const char *)ptr, max_len); unlock_user(ptr, guest_addr, 0); guest_addr +=3D len; /* we don't allow wrapping or integer overflow */ diff --git a/docs/devel/style.rst b/docs/devel/style.rst index d025933808..0b854ae042 100644 --- a/docs/devel/style.rst +++ b/docs/devel/style.rst @@ -519,7 +519,6 @@ QEMU provides other useful string functions: =20 int strstart(const char *str, const char *val, const char **ptr) int stristart(const char *str, const char *val, const char **ptr) - int qemu_strnlen(const char *s, int max_len) =20 There are also replacement character processing macros for isxyz and toxyz, so instead of e.g. isalnum you should use qemu_isalnum. diff --git a/include/qemu/cutils.h b/include/qemu/cutils.h index 36c68ce86c..d249f22676 100644 --- a/include/qemu/cutils.h +++ b/include/qemu/cutils.h @@ -101,22 +101,7 @@ int strstart(const char *str, const char *val, const c= har **ptr); * false otherwise. */ int stristart(const char *str, const char *val, const char **ptr); -/** - * qemu_strnlen: - * @s: string - * @max_len: maximum number of bytes in @s to scan - * - * Return the length of the string @s, like strlen(), but do not - * examine more than @max_len bytes of the memory pointed to by @s. - * If no NUL terminator is found within @max_len bytes, then return - * @max_len instead. - * - * This function has the same behaviour as the POSIX strnlen() - * function. - * - * Returns: length of @s in bytes, or @max_len, whichever is smaller. - */ -int qemu_strnlen(const char *s, int max_len); + /** * qemu_strsep: * @input: pointer to string to parse diff --git a/linux-user/uaccess.c b/linux-user/uaccess.c index 27e841e651..f9bd713edd 100644 --- a/linux-user/uaccess.c +++ b/linux-user/uaccess.c @@ -99,7 +99,7 @@ ssize_t target_strlen(abi_ulong guest_addr1) ptr =3D lock_user(VERIFY_READ, guest_addr, max_len, 1); if (!ptr) return -TARGET_EFAULT; - len =3D qemu_strnlen((const char *)ptr, max_len); + len =3D strnlen((const char *)ptr, max_len); unlock_user(ptr, guest_addr, 0); guest_addr +=3D len; /* we don't allow wrapping or integer overflow */ diff --git a/util/cutils.c b/util/cutils.c index 9803f11a59..f39a97e509 100644 --- a/util/cutils.c +++ b/util/cutils.c @@ -54,7 +54,7 @@ =20 void strpadcpy(char *buf, int buf_size, const char *str, char pad) { - int len =3D qemu_strnlen(str, buf_size); + size_t len =3D strnlen(str, buf_size); memcpy(buf, str, len); memset(buf + len, pad, buf_size - len); } @@ -118,19 +118,6 @@ int stristart(const char *str, const char *val, const = char **ptr) return 1; } =20 -/* XXX: use host strnlen if available ? */ -int qemu_strnlen(const char *s, int max_len) -{ - int i; - - for(i =3D 0; i < max_len; i++) { - if (s[i] =3D=3D '\0') { - break; - } - } - return i; -} - char *qemu_strsep(char **input, const char *delim) { char *result =3D *input; --=20 2.50.1 (Apple Git-155)