<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
On 04/19/2011 01:44 PM, Nathan Kinder wrote:
<blockquote cite="mid:4DADF42A.1080507@redhat.com" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
I've attached a revised patch to address some comments given by
Noriko. The only differences here are in the plug-in close()
function. We free the config lock now, and we also check if the
started flag was set before freeing any of the resources.<br>
<br>
On 04/19/2011 08:52 AM, Nathan Kinder wrote:
<blockquote cite="mid:4DADAFBD.7080004@redhat.com" type="cite"><a
moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://directory.fedoraproject.org/wiki/Auto_Membership_Design">http://directory.fedoraproject.org/wiki/Auto_Membership_Design</a>
<br>
<pre wrap=""><fieldset class="mimeAttachmentHeader"></fieldset>
--
389-devel mailing list
<a moz-do-not-send="true" class="moz-txt-link-abbreviated" href="mailto:389-devel@lists.fedoraproject.org">389-devel@lists.fedoraproject.org</a>
<a moz-do-not-send="true" class="moz-txt-link-freetext" href="https://admin.fedoraproject.org/mailman/listinfo/389-devel">https://admin.fedoraproject.org/mailman/listinfo/389-devel</a></pre>
</blockquote>
<br>
<pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
--
389-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:389-devel@lists.fedoraproject.org">389-devel@lists.fedoraproject.org</a>
<a class="moz-txt-link-freetext" href="https://admin.fedoraproject.org/mailman/listinfo/389-devel">https://admin.fedoraproject.org/mailman/listinfo/389-devel</a></pre>
</blockquote>
Reviewed++.<br>
</body>
</html>