diff options
| author |   Steve French <stfrench@microsoft.com> | 2025-05-04 18:26:45 -0500 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2025-05-06 09:05:15 -0500 |
| commit | d90b023718a17d308d831fde36b3bb6fa3b511e0 (patch) | |
| tree | f04d914442fd64b9467ba9635d819ad2a547e8dc | |
| parent | Linux 6.15-rc5 (diff) | |
| download | wireguard-linux-d90b023718a17d308d831fde36b3bb6fa3b511e0.tar.xz wireguard-linux-d90b023718a17d308d831fde36b3bb6fa3b511e0.zip | |
smb3 client: warn when parse contexts returns error on compounded operation
Coverity noticed that the rc on smb2_parse_contexts() was not being checked
in the case of compounded operations. Since we don't want to stop parsing
the following compounded responses which are likely valid, we can't easily
error out here, but at least print a warning message if server has a bug
causing us to skip parsing the open response contexts.
Addresses-Coverity: 1639191
Acked-by: Paulo Alcantara (Red Hat) <pc@manguebit.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
| -rw-r--r-- | fs/smb/client/smb2inode.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/smb/client/smb2inode.c b/fs/smb/client/smb2inode.c index 57d9bfbadd97..2a3e46b8e15a 100644 --- a/fs/smb/client/smb2inode.c +++ b/fs/smb/client/smb2inode.c @@ -666,6 +666,8 @@ finished: /* smb2_parse_contexts() fills idata->fi.IndexNumber */ rc = smb2_parse_contexts(server, &rsp_iov[0], &oparms->fid->epoch, oparms->fid->lease_key, &oplock, &idata->fi, NULL); + if (rc) + cifs_dbg(VFS, "rc: %d parsing context of compound op\n", rc); } for (i = 0; i < num_cmds; i++) { |