Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 8 Sep 2014 04:51:04 GMT
From:      Christian Brueffer <brueffer@FreeBSD.org>
To:        Perforce Change Reviews <perforce@FreeBSD.org>
Subject:   PERFORCE change 1199465 for review
Message-ID:  <201409080451.s884p4Up026782@skunkworks.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://p4web.freebsd.org/@@1199465?ac=10

Change 1199465 by brueffer@brueffer_freefall on 2014/08/28 12:05:26

	Unlock the right lock.
	
	The adist_remote_lock is not held in this place, whereas the
	adist_recv_list_lock lock is and is picked up during the next iteration.
	
	Submitted by:	ed

Affected files ...

.. //depot/projects/trustedbsd/openbsm/bin/auditdistd/sender.c#5 edit

Differences ...

==== //depot/projects/trustedbsd/openbsm/bin/auditdistd/sender.c#5 (text+ko) ====

@@ -26,7 +26,7 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  *
- * $P4: //depot/projects/trustedbsd/openbsm/bin/auditdistd/sender.c#4 $
+ * $P4: //depot/projects/trustedbsd/openbsm/bin/auditdistd/sender.c#5 $
  */
 
 #include <config/config.h>
@@ -643,7 +643,7 @@
 			 * we can use that.
 			 */
 			if (TAILQ_EMPTY(&adist_recv_list)) {
-				rw_unlock(&adist_remote_lock);
+				mtx_unlock(&adist_recv_list_lock);
 				continue;
 			}
 			mtx_unlock(&adist_recv_list_lock);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201409080451.s884p4Up026782>