]> www.infradead.org Git - users/jedix/linux-maple.git/commit
cifs: do not include page data when checking signature
authorEnzo Matsumiya <ematsumiya@suse.de>
Wed, 18 Jan 2023 17:06:57 +0000 (14:06 -0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 24 Jan 2023 06:24:36 +0000 (07:24 +0100)
commitd048df8baf90c42b29480d8c23e58bd91d53074d
tree9cbc0dd993921c0583b799d980efd94c9d0230d9
parent1004fc90f0d79a4b7d9e3d432729914f472f9ad1
cifs: do not include page data when checking signature

commit 30b2b2196d6e4cc24cbec633535a2404f258ce69 upstream.

On async reads, page data is allocated before sending.  When the
response is received but it has no data to fill (e.g.
STATUS_END_OF_FILE), __calc_signature() will still include the pages in
its computation, leading to an invalid signature check.

This patch fixes this by not setting the async read smb_rqst page data
(zeroed by default) if its got_bytes is 0.

This can be reproduced/verified with xfstests generic/465.

Cc: <stable@vger.kernel.org>
Signed-off-by: Enzo Matsumiya <ematsumiya@suse.de>
Reviewed-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/cifs/smb2pdu.c