diff options
| author | Jeff Layton <jlayton@redhat.com> | 2010-12-07 03:07:33 +0100 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2010-12-07 20:17:59 +0100 |
| commit | 03ceace5c6923ffbcf2b4d2e37afbffbdb5d4a67 (patch) | |
| tree | 5f8630682dbc8e7e0b38c47e1403945b0b5d07d6 /fs/cifs/connect.c | |
| parent | cifs: remove Local_System_Name (diff) | |
| download | linux-03ceace5c6923ffbcf2b4d2e37afbffbdb5d4a67.tar.xz linux-03ceace5c6923ffbcf2b4d2e37afbffbdb5d4a67.zip | |
cifs: fix check of error return from is_path_accessable
This function will return 0 if everything went ok. Commit 9d002df4
however added a block of code after the following check for
rc == -EREMOTE. With that change and when rc == 0, doing the
"goto mount_fail_check" here skips that code, leaving the tlink_tree
and master_tlink pointer unpopulated. That causes an oops later
in cifs_root_iget.
Reported-and-Tested-by: Robbert Kouprie <robbert@exx.nl>
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to '')
| -rw-r--r-- | fs/cifs/connect.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c index 53f9c31a52b2..387b91ae7400 100644 --- a/fs/cifs/connect.c +++ b/fs/cifs/connect.c @@ -2839,7 +2839,7 @@ remote_path_check: goto mount_fail_check; } rc = is_path_accessible(xid, tcon, cifs_sb, full_path); - if (rc != -EREMOTE) { + if (rc != 0 && rc != -EREMOTE) { kfree(full_path); goto mount_fail_check; } |