<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<br>
<div class="moz-cite-prefix">On 04/07/2015 02:41 PM, Pavel Březina
wrote:<br>
</div>
<blockquote cite="mid:5523D08B.30900@redhat.com" type="cite">On
04/03/2015 05:31 PM, Pavel Reichl wrote:
<br>
<blockquote type="cite">
<br>
<br>
On 04/03/2015 01:16 PM, Pavel Březina wrote:
<br>
<blockquote type="cite"><a class="moz-txt-link-freetext" href="https://fedorahosted.org/sssd/ticket/2611">https://fedorahosted.org/sssd/ticket/2611</a>
<br>
<br>
<br>
_______________________________________________
<br>
sssd-devel mailing list
<br>
<a class="moz-txt-link-abbreviated" href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a>
<br>
<a class="moz-txt-link-freetext" href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a>
<br>
</blockquote>
Hello Pavel,
<br>
<br>
you patch LGTM and ci passed, but my tests failed:
<br>
1) enable enumeration
<br>
2) set ldap_enumeration_refresh_timeout to something lower than
default
<br>
3) enter break-point into sdap_get_users_send()
<br>
4) wait for ldap_enumeration_refresh_timeout seconds then
continue
<br>
5) segfault should happen
<br>
<br>
If I apply on top of your patch my attached patch the test
passes, would
<br>
you consider squeezing my patch into yours?
<br>
<br>
Thanks!
<br>
</blockquote>
<br>
Thanks, squashed.
<br>
<br>
</blockquote>
Great, thanks. But now I guess somebody else must do the review as
we both changed the code.<br>
<blockquote cite="mid:5523D08B.30900@redhat.com" type="cite"><br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
sssd-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a>
<a class="moz-txt-link-freetext" href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a>
</pre>
</blockquote>
<br>
</body>
</html>