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 A280733F7; Wed, 21 Feb 2024 17:13:48 +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=1708535631; cv=none; b=VXJiBwY8cg3cdSBIWgfA/LYBKskpKmy/dhMoyT17Uz3Sp5yGiUYK5gSYf9zY+kq9uo2e1wDohuk3qh3boVRke4ivr7GBg4LeMNktjTM/sbLi2JF4f/EAvmYXhGM4Oyso82gyHPoAh1sFvEH/JEO4Q2e3JmKNK7VcTzaiCiWOwCU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708535631; c=relaxed/simple; bh=jggTDE9oGVLyy1ERt5LYHY11LaM/dNOjk89c0ST+Bfw=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=MZN+f4rUa+2t3IvyuIjtXe1oW91OaYss/uK+KMYeu1Ncaa3VCtGf8fE5dzHXHbN/1AoMdNOt/QQOpvJufqs+qwSspueUYilnoBI+GRzbs1/ds7fdXZ2wxPRccdSlrNBJhL+xxRBRqOk/KrREIrFKMVkgYsUtO4ZbMhWEYUs4PaQ= 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 BA4B572C90D; Wed, 21 Feb 2024 20:13:46 +0300 (MSK) Received: from altlinux.org (sole.flsd.net [185.75.180.6]) by imap.altlinux.org (Postfix) with ESMTPSA id AD97136D016F; Wed, 21 Feb 2024 20:13:46 +0300 (MSK) Date: Wed, 21 Feb 2024 20:13:46 +0300 From: Vitaly Chikunov To: Greg Kroah-Hartman Cc: Sasha Levin , stable@vger.kernel.org, Kees Cook , linux-cifs@vger.kernel.org Subject: Re: [PATCH] cifs: Convert struct fealist away from 1-element array Message-ID: <20240221171346.iur7yuuogg2bu3lq@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, 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? I sent the backported patch as you suggested[1] but I don't see it appearing in 6.1.79-rc1 review. Did I make some mistake while sending it? Thanks, [1] https://lore.kernel.org/stable/20240218111538.2592901-1-vt@altlinux.org/ > > thanks, > > greg k-h