Received: by 2002:ab2:b82:0:b0:1f3:401:3cfb with SMTP id 2csp1028729lqh; Fri, 29 Mar 2024 05:16:25 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCUWy8cG/3qlbrCBEueavARbOatd/F1G4SpghL91K7PPvZap0MG3Bm7HoNe33t/QO3UNj3yGR3Mfa49LNZtj/Y6BtomSmOxtGAQVCSyBtQ== X-Google-Smtp-Source: AGHT+IHezVSpF7LM7AoAdFGDehsk6btqC15zFIlDMtqaAVAFQ2cHufURjFjKUyRmH7qs23TdsWNW X-Received: by 2002:a05:6830:1d41:b0:6e6:c9a5:6585 with SMTP id p1-20020a0568301d4100b006e6c9a56585mr2226756oth.23.1711714585053; Fri, 29 Mar 2024 05:16:25 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1711714585; cv=pass; d=google.com; s=arc-20160816; b=UQl5MucUE0XvPjDgGhKzMAtt3FWMOrABiANy5pK2/tir4bKFn6RetxtDfQLRdn/QOr c9fz6BHPBTvj6uatDUFGPu9jTJwCov6dKB9h2JoVJuNoc0CoB6dJQcQGy/4ErjHqcbZd FkH3CUgjqoYlRBNfBn0048j+5lmuD30QV5xMHCUT1OTBT8M8SuOxKFz12BGSNPuoyM3N 4J3tBojff+JDuld04fbMqjuJLawMM1PTxzmut8nzL8Ayn7bAOSJKj3ReVDTx8Ew/AqgX xujLrqVtxfayfmkoBqZ4U9uchXlv8MUfpz00Cx2GgIJeAYhuyZdfSocrZ2Yp+mTL40HI TosA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=6CAYDhhTcC5cDSEI9NxqWNwMQ9HdYMRP1kZOEftfi0Y=; fh=6evdFXumMv2EEJ2jZ5TpypKY+6eqCgy+bZQkYZeTmxg=; b=vqLmLY9DHMxnMqJmOWVhMhJ2GxMXR+i3gFEHP5TsA29dtrqfmLVhUXWlJwjpznO+lJ 2jRESZz2H8IA2tlvo2rSGIdlI8+ULCaep3YrZFx84qnyWmfR06ln136VKstgSVFBF7Ez p00vp5wUVDNp+NJjYTVWkzV3VGlhhUcwrmCjK7g05jqUCqioWGAD1P4ugDqf9bu2wrfz zBZC4fuykGIg3roq0Fd+K/tR8maR6QYsv9gkWD7RIU/kAd7rbCk+B/0cww7EUeCYck74 NP8dKnmNLbOkvwvBW4dilTMQIucBwInaVhXMQM+YMHBCpQbc9HquSh5C81XxYT03NjIO FyFw==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=1FD2lMGV; arc=pass (i=1 dkim=pass dkdomain=linuxfoundation.org); spf=pass (google.com: domain of linux-kernel+bounces-124469-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-124469-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id pa30-20020a05620a831e00b0078a65467f39si3361880qkn.409.2024.03.29.05.16.24 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 29 Mar 2024 05:16:25 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-124469-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=1FD2lMGV; arc=pass (i=1 dkim=pass dkdomain=linuxfoundation.org); spf=pass (google.com: domain of linux-kernel+bounces-124469-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-124469-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id C17651C220FC for ; Fri, 29 Mar 2024 12:16:24 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 85E118594C; Fri, 29 Mar 2024 12:16:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="1FD2lMGV" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9579185926; Fri, 29 Mar 2024 12:16:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711714568; cv=none; b=YmpSGIglk7bWWdCK2mWK/UR+YYU4gFaKc3RLX1PzVO7DaVIlDpuqXlC+7su5Nc8/dbIyz0K0tIPisTlU3LBU/XUyGWFKO5ayiqqcC/PyCIF9t72zOt1NkIF8BEbuCKlzZzI3YqIJe8akQxVkoS0IqITySH3ur3Q2xG6ebmnSGtk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1711714568; c=relaxed/simple; bh=DvKUDWV/EXNQIZ+0TFa/9gUHACM9R6BvIrd7Pb2rvdM=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=D5PMbr7YrkNE+wx9JbzliwsJ18Z1mifO21iWyrVzBz8WAc8FfrCb1GsgCNhTcrzH2PiqWPaTQlThC5R96Op5nWgQB/2Xnwl6Ku5VG1HyfNuEfcBBY7hTipO8O/jcuve4s6YFP7y07Y7pLCAtJ5rNv5iNTwCVq0CkWt2pOF+cuF0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=1FD2lMGV; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id ADB76C433F1; Fri, 29 Mar 2024 12:16:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1711714568; bh=DvKUDWV/EXNQIZ+0TFa/9gUHACM9R6BvIrd7Pb2rvdM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=1FD2lMGVU+MwKTbxTQB2F5/MMKBMSVWKGAOXW4I2ZwM8DxP969Nn9iX30EMxCHYmP MIGH3PLIAazO8V4+zwTa/O+89JJOhJ1KtBCcTlAYcN0+3oNbYEr1mpZG2dCksFX+qi TVzUGHoZrgsbxBL8P93iA97qSEN+429eIdk3l67I= Date: Fri, 29 Mar 2024 13:16:05 +0100 From: Greg KH To: David Sterba Cc: Maximilian Heyne , Goldwyn Rodrigues , Anand Jain , David Sterba , stable@vger.kernel.org, Chris Mason , Josef Bacik , linux-btrfs@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] btrfs: allocate btrfs_ioctl_defrag_range_args on stack Message-ID: <2024032957-muppet-scabby-e708@gregkh> References: <20240320113156.22283-1-mheyne@amazon.de> <20240320161433.GF14596@suse.cz> 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: <20240320161433.GF14596@suse.cz> On Wed, Mar 20, 2024 at 05:14:33PM +0100, David Sterba wrote: > On Wed, Mar 20, 2024 at 11:31:56AM +0000, Maximilian Heyne wrote: > > From: Goldwyn Rodrigues > > > > commit c853a5783ebe123847886d432354931874367292 upstream. > > > > Instead of using kmalloc() to allocate btrfs_ioctl_defrag_range_args, > > allocate btrfs_ioctl_defrag_range_args on stack, the size is reasonably > > small and ioctls are called in process context. > > > > sizeof(btrfs_ioctl_defrag_range_args) = 48 > > > > Reviewed-by: Anand Jain > > Signed-off-by: Goldwyn Rodrigues > > Reviewed-by: David Sterba > > Signed-off-by: David Sterba > > CC: stable@vger.kernel.org # 4.14+ > > [ This patch is needed to fix a memory leak of "range" that was > > introduced when commit 173431b274a9 ("btrfs: defrag: reject unknown > > flags of btrfs_ioctl_defrag_range_args") was backported to kernels > > lacking this patch. Now with these two patches applied in reverse order, > > range->flags needed to change back to range.flags. > > This bug was discovered and resolved using Coverity Static Analysis > > Security Testing (SAST) by Synopsys, Inc.] > > Signed-off-by: Maximilian Heyne > > Acked-by: David Sterba > > for backport to stable as a prerequisite for 173431b274a9a5 ("btrfs: > defrag: reject unknown flags of btrfs_ioctl_defrag_range_args"). > Now queued up, thanks! greg k-h