mirror of
https://github.com/torvalds/linux.git
synced 2024-11-14 08:02:07 +00:00
smb3: add additional null check in SMB311_posix_mkdir
Although unlikely for it to be possible for rsp to be null here, the check is safer to add, and quiets a Coverity warning. Addresses-Coverity: 1437501 ("Explicit Null dereference") Reviewed-by: Paulo Alcantara (SUSE) <pc@cjr.nz> Signed-off-by: Steve French <stfrench@microsoft.com>
This commit is contained in:
parent
9e7ffa77b2
commit
ca780da5fd
@ -2668,7 +2668,18 @@ int smb311_posix_mkdir(const unsigned int xid, struct inode *inode,
|
||||
goto err_free_rsp_buf;
|
||||
}
|
||||
|
||||
/*
|
||||
* Although unlikely to be possible for rsp to be null and rc not set,
|
||||
* adding check below is slightly safer long term (and quiets Coverity
|
||||
* warning)
|
||||
*/
|
||||
rsp = (struct smb2_create_rsp *)rsp_iov.iov_base;
|
||||
if (rsp == NULL) {
|
||||
rc = -EIO;
|
||||
kfree(pc_buf);
|
||||
goto err_free_req;
|
||||
}
|
||||
|
||||
trace_smb3_posix_mkdir_done(xid, le64_to_cpu(rsp->PersistentFileId),
|
||||
tcon->tid,
|
||||
ses->Suid, CREATE_NOT_FILE,
|
||||
|
Loading…
Reference in New Issue
Block a user