| To: | "Christian, Chip" <chip.christian@xxxxxxxxxxxxxxx> |
|---|---|
| Subject: | RE: [NFS] kernel nfsd crash |
| From: | Stephen Walton <swalton@xxxxxxxxxxxxxxxx> |
| Date: | Thu, 9 Aug 2001 14:49:17 -0700 (PDT) |
| Cc: | "'NFS list (E-mail)'" <nfs@xxxxxxxxxxxxxxxxxxxxx>, "Linux XFS (E-mail)" <linux-xfs@xxxxxxxxxxx> |
| In-reply-to: | <23D04BDBA646D411BDDD00D0B774B5390460250A@SA-BWMAIL1> |
| Reply-to: | <swalton@xxxxxxxxxxxxxxxx> |
| Sender: | owner-linux-xfs@xxxxxxxxxxx |
On Thu, 9 Aug 2001, Christian, Chip wrote:
> --- nfsfh.c 2001/05/09 00:54:56 1.1
> +++ nfsfh.c 2001/05/09 00:56:01
> @@ -244,6 +244,10 @@
> */
> pdentry = child->d_inode->i_op->lookup(child->d_inode, tdentry);
> d_drop(tdentry); /* we never want ".." hashed */
> + if (!pdentry && tdentry->d_inode == NULL) {
> + dput(tdentry);
> + pdentry = ERR_PTR(-EINVAL);
> ==> + return()
> + }
Wouldn't this be simpler:
if (!pdentry && tdentry->d_inode == NULL) {
dput(tdentry);
return ERRPTR(-EINVAL);
}
----
Stephen Walton, Professor of Physics and Astronomy,
California State University, Northridge
stephen.walton@xxxxxxxx
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: 3ware driver and XFS kernel from SGI, yocum |
|---|---|
| Next by Date: | Oracle 8.1.7_03 on RH 7.1 XFS 1.0 installer, Gonyou, Austin |
| Previous by Thread: | RE: [NFS] kernel nfsd crash, Christian, Chip |
| Next by Thread: | RE: [NFS] kernel nfsd crash, Christian, Chip |
| Indexes: | [Date] [Thread] [Top] [All Lists] |