Received: by 2002:a05:6a10:16a7:0:0:0:0 with SMTP id gp39csp835399pxb; Thu, 5 Nov 2020 14:26:11 -0800 (PST) X-Google-Smtp-Source: ABdhPJyGCu+x+ruJCsmKLpylhu0xYffVCDcS0UNxF0VNogtsC3S4L863jH3B/jMnN9/tSEmYJhzi X-Received: by 2002:a05:6402:21c7:: with SMTP id bi7mr5091370edb.54.1604615171760; Thu, 05 Nov 2020 14:26:11 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1604615171; cv=none; d=google.com; s=arc-20160816; b=EXxp9Zd0kXiOz7Uir0p8B/3Zt8s6BtSZRWm4RpUfL9WJtLULa1H1JegQzp4TFoyZzT nGUrj6faLpgVXwqbZRsUw7D+nFbX9V+HrYCDhPgdQ2JP0NLpQ5V49m1/9yOtflepxKQ3 txJU4QB1EnHtXs1L2lzMPwuA2x4N7s7op972AaF12M4u5XedL1TG5XBakKG+1rbMYe6T cWRB2Ajj9uSwbqlbc7VvhaMoUYaSXDZ091M0ex+vv9rz0TTLHThFvYbLF2y0TavArXp/ djwJrv8ZzapPNktPvMXYVWaiKwAn9d/rQODZsz4KyP89SM9FRoJolVSW8ACme0Kv8VyN tIgQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature:dkim-filter; bh=2FMTCCr/vM0tqD3rC7rEJQ92CbbjE/dr3HwBH07ZDA0=; b=CcrW0d3Jy5BV+1ufWVZ3KSsLphNZ9kfnJY3j3tcj8uwFkPyYgA8R/OP2AjJw+bjf28 aY8RzdVxfoDSsWWTgm5GzaZGH9n57opKf1WWw53GmgEOlbdojjBH85n+D07HKE5ZP0JN 5rtVzOR2xXawcqz48lZXQWMFqxud6PGgTMiq+TZzbpigmdVKNaQ2GKyTA9QFZ1Ss46+p eqPdePBvWrG8tn61sn+AFjbxMzxJJD2JhjTI8ytZRi/Vr6JO+p9+8qsWFHOrDvR1toSO Cb3F8hk+QGblMvuksetridP0NKU9pQ24vBXGw6IXLy4HsoPbAad3gwTOH5XTdqpqtBYU aRlQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@fieldses.org header.s=default header.b=gYEs0F8M; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id co26si2247281edb.209.2020.11.05.14.25.42; Thu, 05 Nov 2020 14:26:11 -0800 (PST) Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@fieldses.org header.s=default header.b=gYEs0F8M; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732518AbgKEWZf (ORCPT + 99 others); Thu, 5 Nov 2020 17:25:35 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34524 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1732295AbgKEWZf (ORCPT ); Thu, 5 Nov 2020 17:25:35 -0500 Received: from fieldses.org (fieldses.org [IPv6:2600:3c00:e000:2f7::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 19773C0613CF for ; Thu, 5 Nov 2020 14:25:35 -0800 (PST) Received: by fieldses.org (Postfix, from userid 2815) id 2824040BC; Thu, 5 Nov 2020 17:25:34 -0500 (EST) DKIM-Filter: OpenDKIM Filter v2.11.0 fieldses.org 2824040BC DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fieldses.org; s=default; t=1604615134; bh=2FMTCCr/vM0tqD3rC7rEJQ92CbbjE/dr3HwBH07ZDA0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=gYEs0F8MGzNI5AjlZsmbog5ML2JMPU9BA26RcD4LQOWDQzf8rvHhKTJhH34Sl7rXM 16cZEiXXsqzv1EnqGyyhzAF/8dntA4E3mA9K+cZMWqwxOu1oKWbKJIsCRmCgnp/bzK EeDAZqnXg4Y4bSZqqe6HSpA7wrgtXqPNVP+TFORU= Date: Thu, 5 Nov 2020 17:25:34 -0500 From: "J. Bruce Fields" To: Dai Ngo Cc: linux-nfs@vger.kernel.org Subject: Re: [PATCH 0/2] NFSD: Fix use-after-free warning when doing inter-server copy Message-ID: <20201105222534.GG25512@fieldses.org> References: <20201029190716.70481-1-dai.ngo@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201029190716.70481-1-dai.ngo@oracle.com> User-Agent: Mutt/1.5.21 (2010-09-15) Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org Applying for 5.10, thanks! --b. On Thu, Oct 29, 2020 at 03:07:14PM -0400, Dai Ngo wrote: > Observed use-after-free messages in /var/log/messages of destination > server when doing inter-server copy. These come from 2 different places > in the code, one from the nfsd4_cleanup_inter_ssc when nfsd_file_put > is called for the source file and the other from nfs4_put_copy when > it's called from nfsd4_cb_offload_release. > > Fixed by removing the call to nfsd_file_put; the object is not allocated > by nfsd_file_alloc, and by initializing refcount for nfsd4_copy in > nfsd4_do_async_copy. > > fs/nfsd/nfs4proc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) >