Messages in this thread Patch in this message | ![/](/images/icornerl.gif) | | From | NeilBrown <> | Date | Fri, 8 Dec 2006 12:13:41 +1100 | Subject | [PATCH 004 of 18] knfsd: nfsd: make exp_rootfh handle exp_parent errors |
| |
From: J.Bruce Fields <bfields@fieldses.org>
Since exp_parent can fail by returning an error (-EAGAIN) in addition to by returning NULL, we should check for that case in exp_rootfh.
(TODO: we should check that userland handles these errors too.)
Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu> Signed-off-by: Neil Brown <neilb@suse.de>
### Diffstat output ./fs/nfsd/export.c | 4 ++++ 1 file changed, 4 insertions(+)
diff .prev/fs/nfsd/export.c ./fs/nfsd/export.c --- .prev/fs/nfsd/export.c 2006-12-08 12:07:28.000000000 +1100 +++ ./fs/nfsd/export.c 2006-12-08 12:08:20.000000000 +1100 @@ -1104,6 +1104,10 @@ exp_rootfh(svc_client *clp, char *path, path, nd.dentry, clp->name, inode->i_sb->s_id, inode->i_ino); exp = exp_parent(clp, nd.mnt, nd.dentry, NULL); + if (IS_ERR(exp)) { + err = PTR_ERR(exp); + goto out; + } if (!exp) { dprintk("nfsd: exp_rootfh export not found.\n"); goto out; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |