diff options
| author | Steve French <stfrench@microsoft.com> | 2022-07-12 19:43:44 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2022-08-25 12:45:53 +0300 |
| commit | 6a77b03098b1491cbaf23aba3f7e5c981755aef2 (patch) | |
| tree | a056cc0dee2d6ffa40ce11580cfecd1f8db66d7b /fs/nfs_common | |
| parent | 14128541cf81047e60168cb6580604f365b04e60 (diff) | |
| download | linux-6a77b03098b1491cbaf23aba3f7e5c981755aef2.tar.xz | |
smb3: check xattr value length earlier
[ Upstream commit 5fa2cffba0b82336a2244d941322eb1627ff787b ]
Coverity complains about assigning a pointer based on
value length before checking that value length goes
beyond the end of the SMB. Although this is even more
unlikely as value length is a single byte, and the
pointer is not dereferenced until laterm, it is clearer
to check the lengths first.
Addresses-Coverity: 1467704 ("Speculative execution data leak")
Reviewed-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'fs/nfs_common')
0 files changed, 0 insertions, 0 deletions