ChangeSet 1.2310, 2005/03/31 08:58:38-08:00, [EMAIL PROTECTED]

        [PATCH] nfsd4: fix use after put() in cb_recall
        
        nfs4_put_delegation() did a dprintk using a pointer that it had just 
called a
        put() on.  This could cause a dereference of a pointer to freed memory 
in some
        situations.
        
        Signed-off-by: Andy Adamson <[EMAIL PROTECTED]>
        Signed-off-by: J. Bruce Fields <[EMAIL PROTECTED]>
        Signed-off-by: Neil Brown <[EMAIL PROTECTED]>
        Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
        Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>



 nfs4callback.c |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)


diff -Nru a/fs/nfsd/nfs4callback.c b/fs/nfsd/nfs4callback.c
--- a/fs/nfsd/nfs4callback.c    2005-03-31 10:30:06 -08:00
+++ b/fs/nfsd/nfs4callback.c    2005-03-31 10:30:06 -08:00
@@ -541,7 +541,7 @@
                atomic_set(&clp->cl_callback.cb_set, 0);
        /* Success or failure, now we're either waiting for lease expiration
         * or deleg_return. */
-       nfs4_put_delegation(dp);
        dprintk("NFSD: nfs4_cb_recall: dp %p dl_flock %p dl_count %d\n",dp, 
dp->dl_flock, atomic_read(&dp->dl_count));
+       nfs4_put_delegation(dp);
        return;
 }
-
To unsubscribe from this list: send the line "unsubscribe bk-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to