diff options
author | David Teigland <teigland@redhat.com> | 2008-01-14 16:48:58 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2008-01-30 12:04:42 -0500 |
commit | ce5246b972f7514af899a63c0faf831d05ed5ee1 (patch) | |
tree | fe7c58c280bde924f88adf5da2620ecf433293bd | |
parent | 755b5eb8bac90b35dc901465a06081aaad94e9ae (diff) |
dlm: fix possible use-after-free
The dlm_put_lkb() can free the lkb and its associated ua structure,
so we can't depend on using the ua struct after the put.
Signed-off-by: David Teigland <teigland@redhat.com>
-rw-r--r-- | fs/dlm/user.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/dlm/user.c b/fs/dlm/user.c index 4f741546f4bb..eb6164816948 100644 --- a/fs/dlm/user.c +++ b/fs/dlm/user.c | |||
@@ -236,12 +236,12 @@ void dlm_user_add_ast(struct dlm_lkb *lkb, int type) | |||
236 | spin_unlock(&proc->asts_spin); | 236 | spin_unlock(&proc->asts_spin); |
237 | 237 | ||
238 | if (eol) { | 238 | if (eol) { |
239 | spin_lock(&ua->proc->locks_spin); | 239 | spin_lock(&proc->locks_spin); |
240 | if (!list_empty(&lkb->lkb_ownqueue)) { | 240 | if (!list_empty(&lkb->lkb_ownqueue)) { |
241 | list_del_init(&lkb->lkb_ownqueue); | 241 | list_del_init(&lkb->lkb_ownqueue); |
242 | dlm_put_lkb(lkb); | 242 | dlm_put_lkb(lkb); |
243 | } | 243 | } |
244 | spin_unlock(&ua->proc->locks_spin); | 244 | spin_unlock(&proc->locks_spin); |
245 | } | 245 | } |
246 | out: | 246 | out: |
247 | mutex_unlock(&ls->ls_clear_proc_locks); | 247 | mutex_unlock(&ls->ls_clear_proc_locks); |