diff options
author | Steve French <sfrench@us.ibm.com> | 2011-07-25 18:04:32 -0400 |
---|---|---|
committer | Steve French <sfrench@us.ibm.com> | 2011-07-25 18:04:32 -0400 |
commit | e010a5ef95b8b6a12b74b548578f7dcf93564347 (patch) | |
tree | 845def645b231e6d57d0f4df6126401f6f13574b /fs/cifs | |
parent | eaf35b1ea8c12edc5ba8299a8ecfe1efab85101b (diff) |
[CIFS] Redundant null check after dereference
Reviewed-by: Shirish Pargaonkar <shirishp@us.ibm.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs')
-rw-r--r-- | fs/cifs/dir.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/cifs/dir.c b/fs/cifs/dir.c index fa8c21d913bc..8e9d37d44e9d 100644 --- a/fs/cifs/dir.c +++ b/fs/cifs/dir.c | |||
@@ -57,11 +57,6 @@ build_path_from_dentry(struct dentry *direntry) | |||
57 | struct cifs_tcon *tcon = cifs_sb_master_tcon(cifs_sb); | 57 | struct cifs_tcon *tcon = cifs_sb_master_tcon(cifs_sb); |
58 | unsigned seq; | 58 | unsigned seq; |
59 | 59 | ||
60 | if (direntry == NULL) | ||
61 | return NULL; /* not much we can do if dentry is freed and | ||
62 | we need to reopen the file after it was closed implicitly | ||
63 | when the server crashed */ | ||
64 | |||
65 | dirsep = CIFS_DIR_SEP(cifs_sb); | 60 | dirsep = CIFS_DIR_SEP(cifs_sb); |
66 | if (tcon->Flags & SMB_SHARE_IS_IN_DFS) | 61 | if (tcon->Flags & SMB_SHARE_IS_IN_DFS) |
67 | dfsplen = strnlen(tcon->treeName, MAX_TREE_SIZE + 1); | 62 | dfsplen = strnlen(tcon->treeName, MAX_TREE_SIZE + 1); |