Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp3471901pxu; Tue, 8 Dec 2020 13:01:07 -0800 (PST) X-Google-Smtp-Source: ABdhPJwv8KEISlz48UQov3z5y6E9gDHpP85gzcRzOpsayhNTQ2aVOD3JyQDaXxJQtTleRLK7/M6Y X-Received: by 2002:a17:906:33c4:: with SMTP id w4mr23727202eja.380.1607461267357; Tue, 08 Dec 2020 13:01:07 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607461267; cv=none; d=google.com; s=arc-20160816; b=Aki20BUn12/aEkX4DuEPb0j59iYLK/cxdrU4i3gSrYEDIoJGhj9qX/audHzZSrXFIn 4gMQHgERw7ZKyM/+csOaIV/gjOcCx6cOUYP1W1HkkKYGiTTT9YoiEr8G7F3rcRwYqS5F CV6ipYjigw2jx9CXd5feRC6fDGsyarncZYv/1HUcM+tndLfmpljSuBhyGttAppd+sQxd mmiit8hUaMSNB38x0IZnz2XpGDnQpVPb4m4JjgIqlMgXNlemBAHkUlh4uRkecVSRv6Ur QRhSJPM1ZKbnaVt5C9463wxyIO2hOH5fafdsIO+AegIlgX+HAOFAC6qklO0AU/JTbYF1 TyYQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from:sender :dkim-signature; bh=ta0oJLpJ85BMcjzd4A8eoeI6BZCDRtN4qtDaP420o2o=; b=h79cPKf/XbmfSMxEccF2RadJ+LN4X2bAnxniDqH48j3NVFFVqeVgFE2nqXaixyLgGT AsSNg1FxVfoHgOSX7LNzH1CqzgBRM0zH/fsyCqO6Zj64nhFtkrAaykbRNoneR5em5jW7 QWMN5wkqScbygmxWC3SAuhNywddRCwld82C5fKa17F0jhPLJU1e1pdrUUcKW/PmXnJf7 LEkVbfPys5TndXFQMJRhaLO/wxw20r28cSuaTMAvA5VfmWY74d5LGIoG78/sT30lQW5d 15v0FCONcCBPejmKJVyyDvCJLJ53tbKpkVSyreLOgSJOeu3jn8mFy/54ICvHOle4K7AO A8DQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=uGoXH08+; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v26si10628615edw.486.2020.12.08.13.00.42; Tue, 08 Dec 2020 13:01:07 -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=@gmail.com header.s=20161025 header.b=uGoXH08+; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729260AbgLHUaQ (ORCPT + 99 others); Tue, 8 Dec 2020 15:30:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:39344 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727660AbgLHUaP (ORCPT ); Tue, 8 Dec 2020 15:30:15 -0500 Received: from mail-qv1-xf42.google.com (mail-qv1-xf42.google.com [IPv6:2607:f8b0:4864:20::f42]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AF0E8C061793 for ; Tue, 8 Dec 2020 12:29:29 -0800 (PST) Received: by mail-qv1-xf42.google.com with SMTP id p12so8850382qvj.13 for ; Tue, 08 Dec 2020 12:29:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=ta0oJLpJ85BMcjzd4A8eoeI6BZCDRtN4qtDaP420o2o=; b=uGoXH08+Tlrb42quO0VKWOVA5dwX0YDaTUbqQnegeHV/rWCBPtnubrpZqtZAiUATBH w/u+8ES6Kk6j/32MtKr8XPX5SYS01P2heKH2yFmPLFRSAB9cLMpEv7LjFzgnrUtYXtyB 1jPMdW+pbZ9uoc/Vtt+rKa9oiV/gqohSkq9rYbD12YX1bdhiOvTh3TU+2gAnX9056Blb ORY9v8E4yQ0TUQifoVvOp8TxtcstXx69s8GZ9rLh3JZAk6H+iu1ThXlMc6YpEthzdreg ZLkQ3J9wsEvldUVZYIe5LcAr6vCeeVB90HMYBXeLkuNCzRL8N/zLw+PyyZJI5Hh/Ma9W yEKQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references:mime-version:content-transfer-encoding; bh=ta0oJLpJ85BMcjzd4A8eoeI6BZCDRtN4qtDaP420o2o=; b=bsFYcNU+x4THovKLHMoMzJsxdFcHAzGxMLVxfOjGufBREvtyE0ra87YPuUhNtixq73 BJ+QzS1n2mO0+NOLVlTCT1/ysojuW20zrGHO1LGFdNGUPEOSoZqTAc7sOCjc8nCkkwYx Rf3108yIGGeBNSOgLb0DQD6Ex62YSKJhlbBcvAowpiNF6tlBxghs78hDx0cIJG37Osrt C1EUQBggIVounvZWdnJ8HBQJ8H3QLdkO2bnopghzFWOepFUbq558ockE5fGYVJYqVCjZ AIBeKmKy05atSqoaSLsu7IyNkNw/+sB6mGM8P8uiT6BNbWT9gqYOJbEG7fswqMoleUep eOOQ== X-Gm-Message-State: AOAM531laFAAFtVnJEJ551Nz4Gl2hEh2bn+XAZd3z5KBUfWrAaPSR6PZ 6rHUptfm+29+QGjYks4C6GQx2vwG3/w= X-Received: by 2002:a05:6214:144b:: with SMTP id b11mr10430230qvy.5.1607459368698; Tue, 08 Dec 2020 12:29:28 -0800 (PST) Received: from gouda.nowheycreamery.com (c-68-32-74-190.hsd1.mi.comcast.net. [68.32.74.190]) by smtp.gmail.com with ESMTPSA id q20sm5057278qkj.49.2020.12.08.12.29.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Dec 2020 12:29:28 -0800 (PST) Sender: Anna Schumaker From: schumaker.anna@gmail.com X-Google-Original-From: Anna.Schumaker@Netapp.com To: linux-nfs@vger.kernel.org Cc: Anna.Schumaker@Netapp.com Subject: [PATCH v2 2/2] SUNRPC: Check if the buffer has fewer bytes than requested Date: Tue, 8 Dec 2020 15:29:25 -0500 Message-Id: <20201208202925.597663-3-Anna.Schumaker@Netapp.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20201208202925.597663-1-Anna.Schumaker@Netapp.com> References: <20201208202925.597663-1-Anna.Schumaker@Netapp.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org From: Anna Schumaker xdr_expand_hole() might truncate data off the end of the buffer. If that happens, we need to return a short read to the NFS code to indicate that some data has been lost. Fixes: e6ac0accb27c "SUNRPC: Add an xdr_align_data() function" Signed-off-by: Anna Schumaker --- net/sunrpc/xdr.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/sunrpc/xdr.c b/net/sunrpc/xdr.c index 5b848fe65c81..68f470e33427 100644 --- a/net/sunrpc/xdr.c +++ b/net/sunrpc/xdr.c @@ -1273,6 +1273,8 @@ uint64_t xdr_align_data(struct xdr_stream *xdr, uint64_t offset, uint32_t length bytes = xdr->nwords << 2; if (length < bytes) bytes = length; + if (bytes < length) + length = bytes; /* Move page data to the left */ if (from > offset) { -- 2.29.2