浏览代码

fs/ocfs2/dlm: Add missing spin_unlock

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 <julia@diku.dk>
Signed-off-by: Joel Becker <joel.becker@oracle.com>
Julia Lawall 15 年之前
父节点
当前提交
6469272c35
共有 1 个文件被更改,包括 1 次插入0 次删除
  1. 1 0
      fs/ocfs2/dlm/dlmdomain.c

+ 1 - 0
fs/ocfs2/dlm/dlmdomain.c

@@ -1709,6 +1709,7 @@ retry:
 		}
 		}
 
 
 		if (dlm_protocol_compare(&dlm->fs_locking_proto, fs_proto)) {
 		if (dlm_protocol_compare(&dlm->fs_locking_proto, fs_proto)) {
+			spin_unlock(&dlm_domain_lock);
 			mlog(ML_ERROR,
 			mlog(ML_ERROR,
 			     "Requested locking protocol version is not "
 			     "Requested locking protocol version is not "
 			     "compatible with already registered domain "
 			     "compatible with already registered domain "