[PATCH] s4-drs: DsGetReplInfo(), infoType = DS_REPL_INFO_METADATA_2_FOR_OBJ

Erick Nascimento erick.nogueira.nascimento at gmail.com
Thu Mar 25 12:47:04 MDT 2010


There was an issue in one the last two patches.
The added line to selftest/knowfail was commented out, here are the
fixed patches.

Best Regards,
Erick Nogueira do Nascimento
Institute of Computing
Campinas State University

2010/3/24 Erick Nascimento <erick.nogueira.nascimento at gmail.com>:
> Andrew,
>
> Here is one patch that add a line on selftest/knowfail for the
> dsgetinfo test cases.
> The another one just fix the return value for not yet supported infoTypes.
>
> Thanks,
> Erick Nogueira do Nascimento
> Institute of Computing
> Campinas State University
>
> On Fri, Mar 19, 2010 at 9:05 PM, Andrew Bartlett <abartlet at samba.org> wrote:
>> On Fri, 2010-03-19 at 20:36 -0300, Erick Nascimento wrote:
>>> Andrew,
>>>
>>> Here is a patch fixing this. The comments are inside.
>>> I don't know much about the torture testcases purposes, I tough it was
>>> ok to remove the not yet supported infotypes from being called, but
>>> maybe I'm wrong.
>>
>> No, you should instead write up the tests in such a way that they are
>> either not called against Samba4, or (much better) to add them to the
>> known failure file.
>>
>> Tests should be written to pass against Windows 2008 R2, and then
>> knownfail updates made so that when run against Samba4, they don't break
>> the overall test.
>>
>> Andrew Bartlett
>>
>> --
>> Andrew Bartlett                                http://samba.org/~abartlet/
>> Authentication Developer, Samba Team           http://samba.org
>> Samba Developer, Cisco Inc.
>>
>>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-s4-drs-Fix-dsgetinfo-error-checking-and-add-this-tes.patch
Type: text/x-patch
Size: 5825 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20100325/6a9ee8ad/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-s4-drs-Fix-the-return-values-for-unsupported-infotyp.patch
Type: text/x-patch
Size: 3535 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20100325/6a9ee8ad/attachment-0001.bin>


More information about the samba-technical mailing list