[prev in list] [next in list] [prev in thread] [next in thread]
List: samba-technical
Subject: Re: Patch to fix a segfault while duping some security descriptors
From: Nadezhda Ivanova <nivanova () samba ! org>
Date: 2010-06-19 19:49:20
Message-ID: AANLkTinLmDKzSEGg2R1adkxWxl-J5BkkQVsKxmqdN9KE () mail ! gmail ! com
[Download RAW message or body]
Oops, sorry, it appears this is something else. I see no problem with it,
its a nice patch. You can push it if you like, if not, I will do it
tomorrow.
On Sat, Jun 19, 2010 at 10:38 PM, Matthias Dieter Wallnöfer
<mdw@samba.org>wrote:
> Well, I don't see it in.
>
> Nadezhda Ivanova wrote:
>
>> Hi,
>> As I remember I reviewed the patch and Anatoliy pushed it. It should be in
>> already.
>>
>> On Fri, Jun 18, 2010 at 7:44 PM, Matthias Dieter Wallnöfer <mdw@samba.org<mailto:
>> mdw@samba.org>> wrote:
>>
>> Nadia,
>>
>> do we still need this patch? Would you like to push it? Or how to
>> proceed here?
>>
>> Matthias
>>
>>
>> Nadezhda Ivanova wrote:
>>
>> Hi,
>> That's a nice fix, but where is the caller that provided an
>> uninitialized
>> DACL? Maybe that needs fixing too.
>>
>>
>> On Wed, May 26, 2010 at 12:49 AM, Anatoliy Atanasov<
>> anatoliy.atanasov@postpath.com
>> <mailto:anatoliy.atanasov@postpath.com>> wrote:
>>
>>
>>
>> Hi there,
>>
>> I had the same problem :), just waiting for the patch to
>> be approved before
>> pushing it.
>>
>> Thanks,
>> Anatoliy
>> ----- Original Message -----
>>
>> From: samba-technical-bounces@lists.samba.org
>> <mailto:samba-technical-bounces@lists.samba.org><
>>
>> samba-technical-bounces@lists.samba.org
>> <mailto:samba-technical-bounces@lists.samba.org>>
>>
>> To: samba-technical@lists.samba.org
>> <mailto:samba-technical@lists.samba.org><
>> samba-technical@lists.samba.org
>> <mailto:samba-technical@lists.samba.org>>
>>
>> Sent: Wednesday, May 26, 2010 0:04:47 AM (GMT+02:00)
>> Athens, Bucharest,
>>
>> Istanbul
>>
>> Subject: Patch to fix a segfault while duping some
>> security descriptors
>>
>>
>> I found a segfault when duplicating a security
>> descriptor that was
>>
>> missing sacls or dacls. I ran into this when a piece
>> of my code wrote
>> a security descriptor for an LDAP user that had no
>> sacls. After doing
>> that, checking or unchecking the "User cannot change
>> password" option
>> in the AD admin tool would cause samba to crash.
>>
>> I fixed this by checking if the acl was null in
>> security_acl_dup
>> function, and returning NULL if that was the case.
>>
>>
>>
>>
>>
>>
>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic