2010-06-15 23:47:16

by Joel Becker

[permalink] [raw]
Subject: Re: [PATCH 15/17] fs/ocfs2/dlm: Add missing spin_unlock

On Wed, May 26, 2010 at 05:58:53PM +0200, Julia Lawall wrote:
> From: Julia Lawall <[email protected]>
>
> Add a spin_unlock missing on the error path. Unlock as in the other code
> that leads to the leave label.
>
> The semantic match that finds this problem is as follows:
> (http://coccinelle.lip6.fr/)
>
> // <smpl>
> @@
> expression E1;
> @@
>
> * spin_lock(E1,...);
> <+... when != E1
> if (...) {
> ... when != E1
> * return ...;
> }
> ...+>
> * spin_unlock(E1,...);
> // </smpl>
>
> Signed-off-by: Julia Lawall <[email protected]>

This patch is now in the 'fixes' branch of ocfs2.git.

Joel

--

"In the arms of the angel, fly away from here,
From this dark, cold hotel room and the endlessness that you fear.
You are pulled from the wreckage of your silent reverie.
In the arms of the angel, may you find some comfort here."

Joel Becker
Principal Software Developer
Oracle
E-mail: [email protected]
Phone: (650) 506-8127