aboutsummaryrefslogtreecommitdiffstats
path: root/fs/nfsd/nfs4state.c
diff options
context:
space:
mode:
authorNeilBrown <neilb@cse.unsw.edu.au>2005-07-07 20:59:14 -0400
committerLinus Torvalds <torvalds@g5.osdl.org>2005-07-07 21:24:08 -0400
commit0fa822e452084032b8495ca0d8e0199329847815 (patch)
tree5be25dc1f1222d2998a9754e5b5e26d516601b9d /fs/nfsd/nfs4state.c
parent67be431350941765e211eeed237c12def3aaba70 (diff)
[PATCH] nfsd4: fix release_lockowner
We oops in list_for_each_entry(), because release_stateowner frees something on the list we're traversing. Signed-off-by: Andy Adamson <andros@citi.umich.edu> Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu> Signed-off-by: Neil Brown <neilb@cse.unsw.edu.au> Signed-off-by: Andrew Morton <akpm@osdl.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'fs/nfsd/nfs4state.c')
-rw-r--r--fs/nfsd/nfs4state.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c
index 9f9db40b5666..e388c9070de4 100644
--- a/fs/nfsd/nfs4state.c
+++ b/fs/nfsd/nfs4state.c
@@ -3084,7 +3084,12 @@ nfsd4_release_lockowner(struct svc_rqst *rqstp, struct nfsd4_release_lockowner *
3084 * of the lockowner state released; so don't release any until all 3084 * of the lockowner state released; so don't release any until all
3085 * have been checked. */ 3085 * have been checked. */
3086 status = nfs_ok; 3086 status = nfs_ok;
3087 list_for_each_entry(sop, &matches, so_perclient) { 3087 while (!list_empty(&matches)) {
3088 sop = list_entry(matches.next, struct nfs4_stateowner,
3089 so_perclient);
3090 /* unhash_stateowner deletes so_perclient only
3091 * for openowners. */
3092 list_del(&sop->so_perclient);
3088 release_stateowner(sop); 3093 release_stateowner(sop);
3089 } 3094 }
3090out: 3095out: