9p: fix bug in attach-per-user
authorEric Van Hensbergen <ericvh@opteron.homeip.net>
Thu, 7 Feb 2008 01:25:08 +0000 (19:25 -0600)
committerEric Van Hensbergen <ericvh@opteron.homeip.net>
Thu, 7 Feb 2008 01:25:08 +0000 (19:25 -0600)
When a new user attached at a directory other than the root, he would end
up in the parent directory of the cwd.  This was due to a logic error in
the code which attaches the user at the mount point and walks back to the
cwd.  This patch fixes that.

Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
fs/9p/fid.c

index b364da70ff28f048dabcb7029076019eec17dc8d..dfebdbe7440e6f31f627fdd41718cf55c459a55d 100644 (file)
@@ -175,7 +175,7 @@ struct p9_fid *v9fs_fid_lookup(struct dentry *dentry)
        if (!wnames)
                return ERR_PTR(-ENOMEM);
 
-       for (d = dentry, i = n; i >= 0; i--, d = d->d_parent)
+       for (d = dentry, i = (n-1); i >= 0; i--, d = d->d_parent)
                wnames[i] = (char *) d->d_name.name;
 
        clone = 1;
@@ -183,7 +183,7 @@ struct p9_fid *v9fs_fid_lookup(struct dentry *dentry)
        while (i < n) {
                l = min(n - i, P9_MAXWELEM);
                fid = p9_client_walk(fid, l, &wnames[i], clone);
-               if (!fid) {
+               if (IS_ERR(fid)) {
                        kfree(wnames);
                        return fid;
                }