[Ocfs2-devel] ocfs2: bug in dlm_process_recovery_data()
Sunil Mushran
sunil.mushran at oracle.com
Tue Feb 9 09:53:01 PST 2010
Dan Carpenter wrote:
> I noticed a change was merged to dlm_process_recovery_data() so I
> thought you might want to fix this as well.
>
> fs/ocfs2/dlm/dlmrecovery.c
> 1788 list_for_each_entry(lock, tmpq, list) {
> 1789 if (lock->ml.cookie != ml->cookie)
> 1790 lock = NULL;
>
> We dereference lock in list_for_each_entry(). Maybe you want
> list_for_each_entry_safe() or something?
>
> 1791 else
> 1792 break;
> 1793 }
> 1794 if (lock)
> 1795 break;
>
Why? That's only required if we are deleting a list entry.
More information about the Ocfs2-devel
mailing list