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>

diff --git a/fs/smb/client/smb2inode.c b/fs/smb/client/smb2inode.c
index 57d9bfbadd97b2b38cbfc938dafe5e0ec080850e..2a3e46b8e15af6a67b3836440ba0d7823a0ba408 100644 (file)
--- 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++) {