Patch to fix a segfault while duping some security descriptors

[Matthias Dieter Wallnöfer]

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 at 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 at lists.samba.org<
>>>        
>> samba-technical-bounces at lists.samba.org>
>>      
>>> To: samba-technical at lists.samba.org<samba-technical at 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.
>>>        
>>      
>