diff options
| author | Ronnie Sahlberg <lsahlber@redhat.com> | 2017-10-30 13:28:03 +1100 |
|---|---|---|
| committer | Steve French <smfrench@gmail.com> | 2017-10-30 02:11:38 -0500 |
| commit | f74bc7c6679200a4a83156bb89cbf6c229fe8ec0 (patch) | |
| tree | c9a48a5f95603591edbeb175ef6bd8776a8df7a5 /fs | |
| parent | 0b07194bb55ed836c2cc7c22e866b87a14681984 (diff) | |
| download | linux-f74bc7c6679200a4a83156bb89cbf6c229fe8ec0.tar.gz linux-f74bc7c6679200a4a83156bb89cbf6c229fe8ec0.tar.bz2 linux-f74bc7c6679200a4a83156bb89cbf6c229fe8ec0.zip | |
cifs: check MaxPathNameComponentLength != 0 before using it
And fix tcon leak in error path.
Signed-off-by: Ronnie Sahlberg <lsahlber@redhat.com>
Signed-off-by: Steve French <smfrench@gmail.com>
CC: Stable <stable@vger.kernel.org>
Reviewed-by: David Disseldorp <ddiss@samba.org>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/cifs/dir.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/cifs/dir.c b/fs/cifs/dir.c index e702d48bd023..81ba6e0d88d8 100644 --- a/fs/cifs/dir.c +++ b/fs/cifs/dir.c @@ -204,7 +204,8 @@ check_name(struct dentry *direntry, struct cifs_tcon *tcon) struct cifs_sb_info *cifs_sb = CIFS_SB(direntry->d_sb); int i; - if (unlikely(direntry->d_name.len > + if (unlikely(tcon->fsAttrInfo.MaxPathNameComponentLength && + direntry->d_name.len > le32_to_cpu(tcon->fsAttrInfo.MaxPathNameComponentLength))) return -ENAMETOOLONG; @@ -520,7 +521,7 @@ cifs_atomic_open(struct inode *inode, struct dentry *direntry, rc = check_name(direntry, tcon); if (rc) - goto out_free_xid; + goto out; server = tcon->ses->server; |