From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f50.google.com (mail-pj1-f50.google.com [209.85.216.50]) (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 2D2E242AA6 for ; Thu, 2 Apr 2026 01:30:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775093433; cv=none; b=N70Xf+PEm7xPrgfEGF0tdN8QFzuITv0RghYpRCvzq65g7ZMGoRRxVjwDRwVXdccYaAXlEL2+05qwzUe0i1veMwLyo4GHp8xX7FujQblPGBfNeFLEVT+kpDcet40ySugGPmjIRHh5A4k9QPxrNFTCXShPNDf0mP3clJfIkZ6swDI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775093433; c=relaxed/simple; bh=3vs324glrrK0BhxdmJWT952XFD7qBwYWDe2Kc8eF920=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=cD1Lqd5powJ8zunfPVl2cp/nm+D7qu30jARFUP4phoP+J3sXB1X9ZMv3BXpPKv4Ib1wnq3Hc0Yzndaq6lxJvtp3T8a0Cw5FaWT2u3JXBBMZhtOlFbsKFuo55UyXRRf2pQa2wluNo27v2hsyToY3G0lMGoE0XQCuDubLJWANsuHg= 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=qaMHyVhx; arc=none smtp.client-ip=209.85.216.50 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="qaMHyVhx" Received: by mail-pj1-f50.google.com with SMTP id 98e67ed59e1d1-35da1af3e10so276929a91.3 for ; Wed, 01 Apr 2026 18:30:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775093431; x=1775698231; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=A+HUKS/EBQv4QdODV++R3W0y5zmuVAdbUeLNXqcNJKE=; b=qaMHyVhxi9Yl8+auhDfHxDbcguDdlsqQMuANwatkt39af5nwYtdm1GKjQq6DBtIPUm dU/iSxKMTTVtHoXlkgs5giueaH+HLhEwqLle8hp9fw82Wiv0Q9LvoaCf6ul6rGOzaicb N8dcXZvQ6NtbYyjjzfyM573yaYYfV9h/jKrKeTgTJXfpPO/gnhGiAiI1e5dRg9GyYEFQ ivg5bt4jNSeIM17TkMsNx+X+9EdyrQhnDA1mUbLrjO79HvtGSwuaIQdTfPxJxx8+XL0m TPrG3G8swgy2Hsqrbl9Y0lNGwwaZ7RDDMiP+7TzNjmWPNV5K9QoFiFWk157ChAllpjoy CHKg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775093431; x=1775698231; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=A+HUKS/EBQv4QdODV++R3W0y5zmuVAdbUeLNXqcNJKE=; b=PRH8sKxE4GITUcLRJqWcbgPjxioHy91B1Gh6LMl34VU9dSZXSmi+MHyN6B7Bh5qbEj 5oF3wIbtRnC+i3NZzqzIsbldVavaygt49co4M1n1JMMASgWsPn38kvlPU2mU2ddzW4/Y FY0QSchTmqbXLh/PPB0wczBGl4hl8nR8mraCaHbBLTpz4oiVZcohv/bFbfpSLXBXJjg5 X6DF6DWPmtd7ebQSFRXHy6PYNXv+rODpgND4OGwn393MPlxuMcoqHibQYu9tnxZQD8qs Sq10J1QDugNAegbYsvZlCyNwdVudu0fRo4m9QZpxWVpMslOjKqLnQGfWG5IBKHmfeDJ/ 4p2Q== X-Forwarded-Encrypted: i=1; AJvYcCVvROb1ubjbtDlI0y/fmREjhhw+b2vEeF7YjZQc5bmUs1ZXpin3Q+Q8viEVgqZ+0sEzyht9QRvNnLhEXWY=@vger.kernel.org X-Gm-Message-State: AOJu0Yy5uDuYwDeodsT+F5dRlrhS1hTQs8sF+hO+PJ7iMYZm9oe15aG+ hpLAomTQaSPAwJ92u6WwQO5WbzpiaUWEynNpGQfAHFh2WB6R00Jokxvp X-Gm-Gg: ATEYQzwt53qcgtJ09MK5aERKpjFgfBqqdwKuzwlyq6/1R4wTTSbsDitaYi14G2U899t OGrF3rZtCxeo+3UCqK3VVStxwDndt+Jo7FWDaK/1aIPsLGkpumTtSzS69NDuhrN6MFFTn789B+u 8lXVJihiY7xiyFKlltiv/Gu+yurYzcfs8V168bKz0+s69tvUn1NYRTZYGEYTr5eeDaot71Oru4I Y9Vc3sOMR4kreUXgwhT4IlSFXTVugy6s+UD/+Rf3/rR1NhilhQ29Y1t/fzuM9ZIshj8SGP48/Di pPxOxPGdsI66Dt5hw8FJfU5MyLO0WJoug+O0YieitUAl7Jo7MS97BrDZF/DetS4y5zNoCrHCbzw qnXgyoUgDVD9VmVsSu/xNafs29hVumZMr/Qlz9xM5QVa8nyJpl7VtLhcOnh5CfnuAT25y69OW7S LUQ5CPP3UCd50Xf084+XiSyUNgnNX/t0z1QNqI7g== X-Received: by 2002:a17:903:384f:b0:2b2:4ded:954e with SMTP id d9443c01a7336-2b269c47566mr59820555ad.24.1775093431355; Wed, 01 Apr 2026 18:30:31 -0700 (PDT) Received: from fedora ([209.132.188.88]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2b27475ff56sm9794995ad.22.2026.04.01.18.30.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Apr 2026 18:30:30 -0700 (PDT) Date: Thu, 2 Apr 2026 01:30:24 +0000 From: Hangbin Liu To: Jakub Kicinski Cc: Donald Hunter , "David S. Miller" , Eric Dumazet , Paolo Abeni , Simon Horman , Andrew Lunn , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next 3/4] ethtool: strset: check nla_len overflow before nla_nest_end Message-ID: References: <20260331-b4-ynl_ethtool-v1-0-dda2a9b55df8@gmail.com> <20260331-b4-ynl_ethtool-v1-3-dda2a9b55df8@gmail.com> <20260331184637.58b968ef@kernel.org> <20260401174956.1e526940@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260401174956.1e526940@kernel.org> On Wed, Apr 01, 2026 at 05:49:56PM -0700, Jakub Kicinski wrote: > On Wed, 1 Apr 2026 07:27:40 +0000 Hangbin Liu wrote: > > On Tue, Mar 31, 2026 at 06:46:37PM -0700, Jakub Kicinski wrote: > > > On Tue, 31 Mar 2026 11:56:13 +0800 Hangbin Liu wrote: > > > > + if (skb_tail_pointer(skb) - (unsigned char *)strings_attr > U16_MAX) > > > > + goto nla_put_failure; > > > > > > bit ugly, let's add a variant of nla_nest_end() which can return > > > an error on overflow (without the warning from patch 4) ? > > > > I was tried to not touch nla_nest_end() as it is used everywhere. But it makes > > sense to me to add a new function to check this. I'm not very good at naming, > > maybe `nla_nest_end_validate()` ? Or any other name if you have? > > I was thinking nla_nest_end_check() to match the __must_check > attribute. But I also like nla_nest_end_safe() > _validate() is a little long but anything <=5 chars is fine. OK, I will use nla_nest_end_safe() until we have a better name.