From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-dl1-f68.google.com (mail-dl1-f68.google.com [74.125.82.68]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2576C40756D for ; Tue, 24 Mar 2026 15:30:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.68 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774366255; cv=none; b=ItaDgILYnkQNMnejwYWnkJukFJnRFU0xA+LFDOdPLZVE+BZS6fAC4AnANqkGJUX72OCj9c2HPFCV2IcZ339rlDpwbxASMbmgDuE6lvMKZ5SnRd/qGjriGfxfQCN0Wm65INyiim3PoeYF8XkCrWd/vC9lWB9K38iW5QWZ9PrZ9ng= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774366255; c=relaxed/simple; bh=IkRvC5Y3yv7EfUIouHSB7F/B/icLT44ICvi/VwM4Hso=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=TnxPdrkZ9V1or0xdzGFB4qRUbIDxtoPzSXbQRfEykYZrq8HsarO/6KpJE4sbfXcSHJCmIF3i3Kc1chL81oRKXIjI2GtoDeDJinmBJqFQYPWNb3IL9CImEsNOgie7CkqVqjUIQBDJw6VAi8ewlEbLTzLzei6R54uEai0ndjMSRys= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=TXTu5mXb; arc=none smtp.client-ip=74.125.82.68 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="TXTu5mXb" Received: by mail-dl1-f68.google.com with SMTP id a92af1059eb24-127380532eeso2671530c88.1 for ; Tue, 24 Mar 2026 08:30:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1774366249; x=1774971049; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=WiVBh44Ha4wklKoyn3mQlolcP3I3uICA1aXisZpChwo=; b=TXTu5mXbOvgnCld20uPbVAxA90LbgMSKWj4aY/HKBV/gqROu0t4G1KCmZ2aCvNvqKt epK7amemp8fedLeQSshBoA7casRlnmbqXULDvFCPfWXcwN9gyTEC75lV7GwREypB9cDE VWay9GSgNKSatkkoVjhRuz1aj3H5TSuZaTaO9wjqsEhQ+CBFKP/WjgNuWmH80+HUSLwR yXO8vvt20J2zzGzm/lkhwelmWh8UQnbgZC2kCEyw5g4O34Be9y+OTn3arkD4sMn4ZS9Y /70AwpNo1VY2VGpFZhxZxaYS2NdvElrxW/ra3sWja7DQ6w4Lj4KA3jQ14WjzA7crqjZe WgAg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1774366249; x=1774971049; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=WiVBh44Ha4wklKoyn3mQlolcP3I3uICA1aXisZpChwo=; b=mH1ZpVatd9GWYGlMIKTc9Xj8LRAij107fbyKCRAXvVWXzZRLQS0bNWLLea0yQQylC4 6IPybhpC7mYGAiddEkNcDomX8VZxnNHalwtRCcvT/t0nyR7gu+MvxBSUA0soNACO6j6u Vm7n0EfXj6PsW+SzeBh9VXrlkOJ/IntZuUyxNiEtC1a69vTvEWIk/xUAG3lAQL7VpZ6/ ilm1k1ueej1yC+p7dQ0K5+n9vx9QNhX58crhem26PF3AfOLZev96s6nUSMXuxnv3a994 O9PORVfJib5fvD9K5uiRarU2I+slTcE7cjNvhyzIUQi0UWZ0YcDuP4uiPoRRSKkyc4EL MdcQ== X-Forwarded-Encrypted: i=1; AJvYcCU+dgZUseF/K1U/7d4SAr6SeTAJvLwxUKXEFgNCLqCANgvkfwvpTYroh+71pVnShhWCkhKPmhatTzs=@vger.kernel.org X-Gm-Message-State: AOJu0YzS8pKYcjx24/v/FATjznC1LC/wvrOqClzN2rWRUHWJJbOMfUva KxORTYNhVOIB61PGnwiIcsBtIacFY2IfA+Unu0ohgbw3EqLL63RHlDtl X-Gm-Gg: ATEYQzyKH0qkmHy+cllwubRzixhqPK0lNJIHY1e1B6cS8TeEgA/qGN9VJC8Qau1pV3P dk0ztO+/WZ4jcECjz4tJ3N5tyTMJZNXuixdd17WiwJi3QKzA8XwSAh2bMHULDqyhMHG3QrBpqln dm0jPjF0DomxpW5XKQ3IJtZWNQxeZU8I6yMc7x5GhPoxjMDUIxIXrX4ue7fcUx4j10+GTImgIN5 LwYV3icfssAtJkgJq6vY6ICbuy6tDT04ITpA4HUv/LIflugMojAjurCkNBezvpcZMvwV+1RH3JO k4dhQTkTfDIWEpNjzqDq1G/zINlemDIy3pgkIUEOSpPNlXK4O/gf7PTrCVSPV0nMi/n97m5qVkp Yn44EsPb/k/O2//G/qBagHdQLwWaCShGC2l4SidklxoIpIJ2ByU3wKC2t4b+BiLj4CxcAHzKJAt CZrEmHKRSKr8sYYQGlhbftCKYh3TB/NQWqt6hBVKpRyHEOUE2PJRZyiMB3E+Ku0YesiZE81QFfg JPg266n X-Received: by 2002:a05:7022:6709:b0:12a:6c84:601f with SMTP id a92af1059eb24-12a726dee93mr8317819c88.36.1774366249330; Tue, 24 Mar 2026 08:30:49 -0700 (PDT) Received: from localhost.localdomain ([38.244.25.197]) by smtp.gmail.com with ESMTPSA id a92af1059eb24-12a736b952asm11668865c88.12.2026.03.24.08.30.45 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Tue, 24 Mar 2026 08:30:48 -0700 (PDT) From: Eric-Terminal To: horms@kernel.org, davem@davemloft.net, kuba@kernel.org Cc: netdev@vger.kernel.org, bridge@lists.linux.dev, linux-nfs@vger.kernel.org, Yufan Chen Subject: [PATCH v3 2/2] net: sunrpc: replace deprecated simple_strtol with kstrtouint Date: Tue, 24 Mar 2026 23:30:36 +0800 Message-ID: <20260324153036.86901-3-ericterminal@gmail.com> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260324153036.86901-1-ericterminal@gmail.com> References: <20260324153036.86901-1-ericterminal@gmail.com> Precedence: bulk X-Mailing-List: linux-nfs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Developer-Signature: v=1; a=openpgp-sha256; l=2022; i=ericterminal@gmail.com; h=from:subject; bh=6agocKJaIItyp0/DYq8x5bAU5b3NaTnGQ7RmsnHGIwk=; b=owGbwMvMwCXWM/dCzeS3H+sZT6slMWQeWqt142ldbiBb3nGxexem6LS+7Hnxdq9vf/LzfVUZT 5nv3p2b3jGRhUGMi8FSTJHl7v99c3O9bs25zn04F2YOKxPIEGmRBgYgYGHgy03MKzXSMdIz1TbU MzTSMdAxZuDiFICpDjzC8D//4YOUx4Y/r2h2zbm/a9vDWZO7Vn2bfVc+sG7JQ7GMaROfMfz3nsO 2NCFps1Lc2d3NjLcjXQzm9DgUTRafp86w+9q1i0kcAA== X-Developer-Key: i=ericterminal@gmail.com; a=openpgp; fpr=DDFFBE9D6D4ADA9CD70BC36D8C9DD07C93EDF17F Content-Transfer-Encoding: 8bit From: Yufan Chen In proc_dodebug(), trim trailing whitespace and use kstrtouint() for full-token conversion, preserving acceptance of surrounding whitespace while rejecting malformed input. This replaces the deprecated simple_strtol(), improves error reporting consistency, and avoids partially parsed values in control paths. Signed-off-by: Yufan Chen --- v3: - Split from mixed series into a dedicated net series. - No functional changes since v2. net/sunrpc/sysctl.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/net/sunrpc/sysctl.c b/net/sunrpc/sysctl.c index bdb587a72..07072218b 100644 --- a/net/sunrpc/sysctl.c +++ b/net/sunrpc/sysctl.c @@ -12,6 +12,7 @@ #include #include #include +#include #include #include @@ -65,10 +66,11 @@ static int proc_dodebug(const struct ctl_table *table, int write, void *buffer, size_t *lenp, loff_t *ppos) { - char tmpbuf[20], *s = NULL; + char tmpbuf[20]; char *p; unsigned int value; size_t left, len; + int ret; if ((*ppos && !write) || !*lenp) { *lenp = 0; @@ -89,19 +91,17 @@ proc_dodebug(const struct ctl_table *table, int write, void *buffer, size_t *len if (left > sizeof(tmpbuf) - 1) return -EINVAL; memcpy(tmpbuf, p, left); + + while (left && isspace(tmpbuf[left - 1])) + left--; tmpbuf[left] = '\0'; + if (!tmpbuf[0]) + goto done; - value = simple_strtol(tmpbuf, &s, 0); - if (s) { - left -= (s - tmpbuf); - if (left && !isspace(*s)) - return -EINVAL; - while (left && isspace(*s)) { - left--; - s++; - } - } else - left = 0; + ret = kstrtouint(tmpbuf, 0, &value); + if (ret) + return ret; + left = 0; *(unsigned int *) table->data = value; /* Display the RPC tasks on writing to rpc_debug */ if (strcmp(table->procname, "rpc_debug") == 0) -- 2.47.3