Commit 44a48081 authored by Steve French's avatar Steve French

smb3: remove unneeded null check in cifs_readdir

Coverity pointed out an unneeded check.

Addresses-Coverity: 1518030 ("Null pointer dereferences")
Reviewed-by: default avatarRonnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: default avatarSteve French <stfrench@microsoft.com>
parent de3a9e94
...@@ -1105,10 +1105,8 @@ int cifs_readdir(struct file *file, struct dir_context *ctx) ...@@ -1105,10 +1105,8 @@ int cifs_readdir(struct file *file, struct dir_context *ctx)
* find_cifs_entry in case there will be reconnects during * find_cifs_entry in case there will be reconnects during
* query_directory. * query_directory.
*/ */
if (cfid) { close_cached_dir(cfid);
close_cached_dir(cfid); cfid = NULL;
cfid = NULL;
}
cache_not_found: cache_not_found:
/* /*
......
...@@ -770,6 +770,7 @@ smb2_cached_lease_break(struct work_struct *work) ...@@ -770,6 +770,7 @@ smb2_cached_lease_break(struct work_struct *work)
/* /*
* Open the and cache a directory handle. * Open the and cache a directory handle.
* Only supported for the root handle. * Only supported for the root handle.
* If error then *cfid is not initialized.
*/ */
int open_cached_dir(unsigned int xid, struct cifs_tcon *tcon, int open_cached_dir(unsigned int xid, struct cifs_tcon *tcon,
const char *path, const char *path,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment