From patchwork Fri Jul 21 14:29:09 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jeff Layton X-Patchwork-Id: 12413 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:9010:0:b0:3e4:2afc:c1 with SMTP id l16csp255295vqg; Fri, 21 Jul 2023 07:45:31 -0700 (PDT) X-Google-Smtp-Source: APBJJlF/0xz3bTq0O1z5HtQm/snFJeZmT9hLrC9aCM9LxuVuD4/VlsEdcDHU5A55s46KjvaFD1Ql X-Received: by 2002:a05:6402:120b:b0:51d:acda:2f4a with SMTP id c11-20020a056402120b00b0051dacda2f4amr1743287edw.37.1689950731362; Fri, 21 Jul 2023 07:45:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1689950731; cv=none; d=google.com; s=arc-20160816; b=087TmYE9RsqTQ14AddGTG2JplMfVZ/K03WVTfpaxYgWqAHUHfEXPno8rBU/CFuyt3a OpU3S1PjwMi9ZG+hN3EFmqkEKmerKB61t/XRFQNFBoNftmW5RaFqvXpfUa9dxyvlQnbx yhQDu3jsMzOfxNpranjVIBICy2xAxn3xfNOEDz2+xlcbnTwEIkWRlmZwWDOxPc25XGOo oTYJ1PKCVJJQ1IOYIClJn+BhW3DpUNUs/4pSTweQxqBZSK9qSlrYQFhyYJW8/Td+t5Ie OKLjz4OwO4wXBstHj+rpQS1bT+JkXnI0Ls8z9LwKln9pFUEEgxxfZEN39QYx5ULPUX7g QgWQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:content-transfer-encoding:mime-version :message-id:date:subject:from:dkim-signature; bh=gRKM2I+n2ATyPA+1Kab1DlmnMN+HftQW+f8D5lUf2z8=; fh=+d9EA0ASG1ukkd7H8IyPKHx6rAxJS58s7dr6FWKuwS8=; b=SP6aw7nMzb1rusJq0bGcTTO4lxj2T7tfi1ix/pjP7h81m7ib4pfEkybO8oCIMa9aTd dAoW8YdhnxmweAPvLmHI7isbqKbxIQK8tYoiWANYANwK9bQyli6ItzG2cuJQsVEkjKwI N5jyyvFcyzvRHPLQhif3u9KWN/W0Wj/i8+z1I5cIE76JHJJWajK2ngc2OI/xv9IOXAKk ae50l1Vb8d7btmVTvQHKmc1M6l8cW+8nEWeuU7i3rRYgSaMpC/gwmp3NK7q50TN8qXqn JNboAdXUt5ALAYlx/A5Xcn7jS6JNubrCIV4QxoDj8KQwUQCGYwGYITcXsK5LPzI/cFqv 5g3A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=AZrO9UGH; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id f23-20020a05640214d700b0051df5d3d1ecsi2417661edx.254.2023.07.21.07.45.04; Fri, 21 Jul 2023 07:45:31 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=AZrO9UGH; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230353AbjGUO3V (ORCPT + 99 others); Fri, 21 Jul 2023 10:29:21 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48834 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229487AbjGUO3S (ORCPT ); Fri, 21 Jul 2023 10:29:18 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 85ABFE68; Fri, 21 Jul 2023 07:29:17 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 23B9161CD0; Fri, 21 Jul 2023 14:29:17 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C4DA0C433C7; Fri, 21 Jul 2023 14:29:15 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1689949756; bh=Ob99k3K/cvLpUD5huHKwap1tAWzVKc9is1OGALrmBu0=; h=From:Subject:Date:To:Cc:From; b=AZrO9UGHCw14XmIIsBlaAKON6cd5enNQzetcn9tg/wibmHBzA2IZOwycxsaVOFTm5 4ekus7dN17JQt1iW7ICyi9b5nqWa1q0mqC2IXkwyFpH+Sx1nfwyOYqdTA7jtbzld0f 0qiV5rgEBcSMkhD3DB6OcvZo9k0oGkQ6Trp//AQbS96RL8GaEHpd69oSnJnMU4UHfD Sh6qURMA5lWZQXJ3t3OqegZIio/wglTZ7uB5uMEyG4VHiVZMJXhvLdFmTsOP4lXLio ztzcQRUu8J6b8w/TfW4c5J9sYK67aDabz/xZnPSdrftVXFHkPMikXDaFT2Ak4mHRxT 823ejTQwXonYA== From: Jeff Layton Subject: [PATCH v3 0/2] nfsd: sanely handle inabilty to fetch pre/post attributes Date: Fri, 21 Jul 2023 10:29:09 -0400 Message-Id: <20230721-bz2223560-v3-0-bb57e302bab7@kernel.org> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIADWWumQC/23MQQ6CMBCF4auQWVsznVagrryHYUFhhEZDSWsal fTuVtYu/5eXb4PIwXGEc7VB4OSi80sJdahgmPtlYuHG0kBIChtCYT9EpE41CjPo2mDfkmwslP8 a+OZeu3XtSs8uPn1473SSv/WfkqSQgkerDSJqZdvLncPCj6MPE3Q55y8hNA2PowAAAA== To: Chuck Lever , Neil Brown , Olga Kornievskaia , Dai Ngo , Tom Talpey Cc: linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org, Jeff Layton , Boyang Xue X-Mailer: b4 0.12.3 X-Developer-Signature: v=1; a=openpgp-sha256; l=1393; i=jlayton@kernel.org; h=from:subject:message-id; bh=Ob99k3K/cvLpUD5huHKwap1tAWzVKc9is1OGALrmBu0=; b=owEBbQKS/ZANAwAIAQAOaEEZVoIVAcsmYgBkupY7dXGHou+JfJYbXGQdoDkov7ilmKJj11NEZ pYWrAIknlGJAjMEAAEIAB0WIQRLwNeyRHGyoYTq9dMADmhBGVaCFQUCZLqWOwAKCRAADmhBGVaC FXlsD/9XQgqQsZoBuU73d8nub8b0vC9JhWBQWIB4yEScl8n2rmlLwqrsg6+cyd3v8E8FpXSiHAZ GvTRYtPGW/jqin8PfNLzCEYi+6BMCrxci8hlZjK3r6TqqmB8qPMDb0t++dcZUjj1ouKX32yZrX4 dQH0TWSBIw9orUl1A4bN0Djw7GxLShy3TvUkN244kc21TMVq2/8Z7ryTIffLs5BfZeQ2xErvFX1 hdu3922N0xh+O+1W94UtyHPJZD+q8yQSWHH8gK/Y12ZKTiSaK2ateliKoLcXl9YsCRG6nBnV+ql Z+5IDE+A2oMtzst49xOx9elK61PSZRHI7jY0Cij/EFdyOqblDtf/611QEjsYcwEO31h/V/g2d2U MpjxAkP9ULKFoRXYTGFzR9UtOVXvWtY0spHng4l6y7XmIoscwvpD8W7PEby68FaQQg2cLHh9l7D nLWz6CL8gSXUH4FxHBzkQfvHlDAoPMiEMFyg5hlIhFlDi4lyDmXOA9g1V67DLTVe185DMQ+/E/s of0FNnFCWCS7/h1g1weLgntPj2TBd4esYifLoe3W0d7aASyaLJSFzQ3cdlXxDZhzdnsFSF8W8Sv OghuGpO93gKoBO+cg+Y0oUawf4bIjBswMhvabjet/ldnGj/AZWW9yyMwQdd4gIX5Nm3zfpnQGCx w6HkX/R3oIiA5rw== X-Developer-Key: i=jlayton@kernel.org; a=openpgp; fpr=4BC0D7B24471B2A184EAF5D3000E684119568215 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1771965481581333667 X-GMAIL-MSGID: 1772041778319657777 Boyang reported tripping the BUG_ON in set_change_info. While we couldn't confirm it, one way this could happen would be for nfsd_lookup to succeed and then for fh_fill_both_attrs to fail. This patchset attempts to (sanely) fix this, usually by aborting the operation if fetching the pre attributes fails. Post-op attribute fetch handling is more difficult to deal with however since we've already done the operation, so this has it just fudge the change_info4 if that occurs. Signed-off-by: Jeff Layton --- Changes in v2: - rework the error paths to consistently use gotos - add __must_check to fh_fill_pre_attrs and fh_fill_both_attrs - fix bad error handling in setxattr codepath --- Jeff Layton (2): nfsd: handle failure to collect pre/post-op attrs more sanely nfsd: remove unsafe BUG_ON from set_change_info fs/nfsd/nfs3proc.c | 4 +++- fs/nfsd/nfs4proc.c | 46 ++++++++++++++++++++++++++++++++++++++++------ fs/nfsd/nfsfh.c | 26 ++++++++++++++++---------- fs/nfsd/nfsfh.h | 6 +++--- fs/nfsd/vfs.c | 52 +++++++++++++++++++++++++++++++++++----------------- fs/nfsd/xdr4.h | 11 ----------- 6 files changed, 97 insertions(+), 48 deletions(-) --- base-commit: c9194156c1039499533303fc63a66b0f1399896b change-id: 20230720-bz2223560-9c4690a8217b Best regards,