ACL module patches

Nadezhda Ivanova nivanova at samba.org
Tue Jul 6 13:56:20 MDT 2010


Oh, right, its added above the previous check. Looks great, give it a push
:)

On Tue, Jul 6, 2010 at 10:53 PM, Matthias Dieter Wallnöfer <mdw at samba.org>wrote:

> Nadya,
>
> I did only add a check that means: if the control was specified then please
> handle this the same way as if we would have one delete and one add password
> mod (password modify operation). If the control isn't specified then do
> perform the other checks which you've introduced.
>
> Matthias
>
> Nadezhda Ivanova wrote:
>
>> Just one quick question - obviously I do not understand this control, but
>> are you sure we should perform the access check only if the control has been
>> specified?
>>
>> Regards,
>> Nadya
>>
>> On Tue, Jul 6, 2010 at 8:50 PM, Nadezhda Ivanova <nivanova at samba.org<mailto:
>> nivanova at samba.org>> wrote:
>>
>>    Hi Matthias,
>>    I basically made sure to do what you did in password_hash, and I
>>    did not see this control there, but it looks all right. As for the
>>    memory context patches - I am not so sure. They will not have any
>>    effect but as far as I know contexts should be as granular as
>>    possible, otherwise you risk a method freeing a context that was
>>    passed as parameter instead of the local one. And the habit of
>>    moving line endings around even if they do not exceed the
>>    character limit... Well, in short, I am not convinced that the
>>    second patch is necessary, but I am not against applying it.
>>    However, since they are your patches, I think you should push them
>>    :). Just make sure to run both acl.py and password.py tests before
>>    that.
>>
>>    Regards,
>>    Nadya
>>
>>
>>    On Tue, Jul 6, 2010 at 7:55 PM, Matthias Dieter Wallnöfer
>>    <mdw at samba.org <mailto:mdw at samba.org>> wrote:
>>
>>        Hi Nadya,
>>
>>        the ACL password work does work well beside one exception of
>>        which you probably wasn't aware. I put here the link to the
>>        patch from which the commit message should explain the reason:
>>
>> http://repo.or.cz/w/Samba/mdw.git/commitdiff/265a2ea034ceb2c29d075933516bbdf6f042c9c9
>>
>>        The other patch should fix the outstanding memory contexts:
>>
>> http://repo.or.cz/w/Samba/mdw.git/commitdiff/b3880b5f25b88b524cbb2b5768e37b83109976a1
>> .
>>        Would be nice if you could integrates this one too.
>>
>>        Greets,
>>        Matthias
>>
>>
>>
>>
>


More information about the samba-technical mailing list