From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from vmicros1.altlinux.org (vmicros1.altlinux.org [194.107.17.57]) by smtp.subspace.kernel.org (Postfix) with ESMTP id BEED92C85C; Fri, 8 Mar 2024 08:48:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=194.107.17.57 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709887729; cv=none; b=rk041+5s0EpiUaPbN3FrrGFjj56mDCUA6tGNR7rPOP1rOx5RGqMdxLx6aW+996Or2EF6qvfn9xlaIf5BGLeD2riJ4cwSp7sTvDQ3k5E9Lzlg7h+X2HcwFbadjxTumZWwI/KFIml+b50WS3vCpyRmYLl8wdwqnQdYGclehmV9II0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709887729; c=relaxed/simple; bh=2EhPOInGWCrsY5uTbTdzHscFzz1mz3zEHPzsntxauqg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=HKZKB798hGPW+18SZuNCB9fUOvbwIcoALYSxt8olkC4FuhqpzNt1uUvj4GIOHXgPIn/UszNUIhc9/KIsfzPn+GBZ5vXmzWZT4KEhpqyrb3GmvB0Y4P882N1omi//CaWPVo1CENVMBZx/sck1paKmbJwKeOSKgq9F6EMgIfb9y5g= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=altlinux.org; spf=pass smtp.mailfrom=altlinux.org; arc=none smtp.client-ip=194.107.17.57 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=altlinux.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=altlinux.org Received: from imap.altlinux.org (imap.altlinux.org [194.107.17.38]) by vmicros1.altlinux.org (Postfix) with ESMTP id 8965672C8CC; Fri, 8 Mar 2024 11:48:44 +0300 (MSK) Received: from altlinux.org (sole.flsd.net [185.75.180.6]) by imap.altlinux.org (Postfix) with ESMTPSA id 7AB4036D0168; Fri, 8 Mar 2024 11:48:44 +0300 (MSK) Date: Fri, 8 Mar 2024 11:48:44 +0300 From: Vitaly Chikunov To: Greg Kroah-Hartman , Sasha Levin , stable@vger.kernel.org Cc: Kees Cook , linux-cifs@vger.kernel.org Subject: Re: [PATCH] cifs: Convert struct fealist away from 1-element array Message-ID: <20240308084844.xo333plkzxtssbqj@altlinux.org> References: <20230215000832.never.591-kees@kernel.org> <202402091559.52D7C2AC@keescook> <20240210003314.jyrvg57z6ox3is5u@altlinux.org> <2024021034-populace-aerospace-03f3@gregkh> <20240210102145.p4diskhnevicn6am@altlinux.org> <20240217215016.emqr3stdm3yrh4dq@altlinux.org> <2024021808-coach-wired-41cb@gregkh> Precedence: bulk X-Mailing-List: linux-cifs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <2024021808-coach-wired-41cb@gregkh> Greg, Sasha, Ping. On Sun, Feb 18, 2024 at 10:31:29AM +0100, Greg Kroah-Hartman wrote: > On Sun, Feb 18, 2024 at 12:50:16AM +0300, Vitaly Chikunov wrote: > > On Sat, Feb 10, 2024 at 01:21:45PM +0300, Vitaly Chikunov wrote: > > > On Sat, Feb 10, 2024 at 10:19:46AM +0000, Greg Kroah-Hartman wrote: > > > > On Sat, Feb 10, 2024 at 03:33:14AM +0300, Vitaly Chikunov wrote: > > > > > > > > > > Can you please backport this commit (below) to a stable 6.1.y tree, it's > > > > > confirmed be Kees this could cause kernel panic due to false positive > > > > > strncpy fortify, and this is already happened for some users. > > > > > > > > What is the git commit id? > > > > > > 398d5843c03261a2b68730f2f00643826bcec6ba > > > > Can you please apply this to the next 6.1.y release? > > > > There is still non-theoretical crash as reported in > > https://lore.kernel.org/all/qjyfz2xftsbch6aozgplxyjfyqnuhn7j44udrucls4pqa5ey35@adxvvrdtagqf/ > > > > If commit hash was not enough: > > > > commit 398d5843c03261a2b68730f2f00643826bcec6ba > > Author: Kees Cook > > AuthorDate: Tue Feb 14 16:08:39 2023 -0800 > > > > cifs: Convert struct fealist away from 1-element array > > > > The commit is in mainline and is applying well to linux-6.1.y: > > > > (linux-6.1.y)$ git cherry-pick 398d5843c03261a2b68730f2f00643826bcec6ba > > Auto-merging fs/smb/client/cifspdu.h > > Auto-merging fs/smb/client/cifssmb.c > > [linux-6.1.y 4a80b516f202] cifs: Convert struct fealist away from 1-element array > > Author: Kees Cook > > Date: Tue Feb 14 16:08:39 2023 -0800 > > 2 files changed, 10 insertions(+), 10 deletions(-) > > It does not apply cleanly due to renames, can you provide a backported, > and tested, patch please? Can you explain please why the patch submission [1] is silently not accepted so I could possibly resubmit it? [1] https://lore.kernel.org/stable/20240218111538.2592901-1-vt@altlinux.org/ It's tested to compile well and to fix the real crash. I also think the submission conforms to Option 3 of process/stable-kernel-rules.rst so I would be glad to know why if it isn't. Thanks, > > thanks, > > greg k-h