xfs
[Top] [All Lists]

RE: [NFS] kernel nfsd crash

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>