[SSSD] [PATCH] Wrong paramater to sysdb_attrs_add_uint32

Jakub Hrozek jhrozek at redhat.com
Fri Jul 29 06:50:49 UTC 2011


On 07/28/2011 06:32 PM, Stephen Gallagher wrote:
> On Thu, 2011-07-28 at 17:56 +0200, Jakub Hrozek wrote:
>> While reviewing Jan's sysdb patches I noticed that a call to
>> sysdb_attrs_add_uint32() I added recently is wrong.
>>
>> The reason the code worked and the compiler did not complain was simple
>> - there's only one place that calls sdap_store_group_with_gid() and it
>> passes group_attrs for mem_ctx. If we ever reused
>> sdap_store_group_with_gid() with a different memory context it would
>> probably crash and burn.
>>
>> Sorry about the inconvenience.
> 
> Ack
> 

Stephen asked me on IRC but I was away already - this belongs to the 1.5
branch as well.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 262 bytes
Desc: OpenPGP digital signature
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110729/f9d28a54/attachment.sig>


More information about the sssd-devel mailing list