<br /><br /><span>Am 01.04.14 schrieb <b class="name">Noriko Hosoi </b> <nhosoi@redhat.com>:</span><blockquote cite="mid:533AED52.6080302@redhat.com" class="iwcQuote" style="border-left: 1px solid #00F; padding-left: 13px; margin-left: 0;" type="cite"><div class="mimepart text html"><span><p>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type" />
<table><tbody><tr><td bgcolor="#FFFFFF" text="#000000"><p>
</p><div class="moz-cite-prefix">Hello Carsten,<br />
<br />
Could it be possible to share your failed test cases with us for
our debugging?</div></td></tr></tbody></table></p></span></div></blockquote>I have setup an winsync replication agreement with a W2K8R2 AD. There is enabled the SFU for managing the Posix attributes uidnumber, gidnumber, unixshell, unixhomedir, nisdomain, ...<br />The test failes if I change some of the Posix attributes. These changes will not synched.<br />The reason is that for #716980 the value for rawentry is reseted before winsync cb is called and for #47314 the cb <span>posix_winsync_pre_ds_mod_user_cb returns immediately if rawentry is NULL.<br />Before #47314 only account enable/disable have to fail.<br /><br />sync_acct_disable I took from ipa.<br />I don't know exactly what is the difference beteween ad_entry and rawentry. Perhaps all works as before if I ignore rawentry in the cb code and use ad_entry instead.<br /><br />Carsten<br /></span><blockquote cite="mid:533AED52.6080302@redhat.com" class="iwcQuote" style="border-left: 1px solid #00F; padding-left: 13px; margin-left: 0;" type="cite"><div class="mimepart text html"><span><p><table><tbody><tr><td bgcolor="#FFFFFF" text="#000000"><div class="moz-cite-prefix"><br />
Thanks,<br />
--noriko<br />
<br />
Rich Megginson wrote:<br />
</div>
<blockquote cite="mid:533AE2EC.90205@redhat.com" type="cite">
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type" />
<div class="moz-cite-prefix">On 04/01/2014 07:34 AM, Carsten
Grzemba wrote:<br />
</div>
<blockquote cite="mid:bbb0ed5253f5.533adc89@contac-dt.de" type="cite">
<div class="commit-subject">Hi Rich,<br />
<br />
this breaks the current implementaion for posix-winsync:<br />
<br />
Bug 716980 - winsync uses old AD entry if new one not found<span class="decoration"></span></div>
<a class="moz-txt-link-freetext" href="https://bugzilla.redhat.com/show_bug.cgi?id=716980" moz-do-not-send="true" target="1">https://bugzilla.redhat.com/show_bug.cgi?id=716980</a>
<br />
Resolves: bug 716980 <br />
Bug Description: winsync uses old AD entry if new one not found
Reviewed by: nhosoi (Thanks!) <br />
Branch: master <br />
Fix Description: Clear out the old raw_entry before doing the
search. This will leave a NULL in the raw entry. winsync plugins
will need to handle a NULL for the raw_entry and/or ad_entry.<br />
<br />
In the moment posix_winsync_pre_ds_mod_user_cb returns
imediataly on raw_entry == NULL<br />
How should the plugin handle the NULL for raw_entry?<br />
</blockquote>
<br />
Not sure. Please reopen that ticket. If it broke posix-winsync,
it is likely to break other winsync plugins (e.g. ipa winsync).<br />
<br />
<blockquote cite="mid:bbb0ed5253f5.533adc89@contac-dt.de" type="cite"><br />
Carsten <br />
<fieldset class="mimeAttachmentHeader"></fieldset>
<br />
<pre wrap="">--
389-devel mailing list
<a 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" moz-do-not-send="true" target="1">https://admin.fedoraproject.org/mailman/listinfo/389-devel</a></pre>
</blockquote>
<br />
<br />
<fieldset class="mimeAttachmentHeader"></fieldset>
<br />
<pre wrap="">--
389-devel mailing list
<a 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" target="1">https://admin.fedoraproject.org/mailman/listinfo/389-devel</a></pre>
</blockquote>
<br />
</td></tr></tbody></table>
</p></span></div></blockquote>