[SSSD] [PATCH] Wrong paramater to sysdb_attrs_add_uint32

Stephen Gallagher sgallagh at redhat.com
Mon Aug 1 11:29:03 UTC 2011


On Fri, 2011-07-29 at 08:50 +0200, Jakub Hrozek wrote:
> 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.

Pushed to master and sssd-1-5.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20110801/3dd52016/attachment.sig>


More information about the sssd-devel mailing list