On Tue, 2012-05-22 at 11:14 +0200, Jakub Hrozek wrote:
> On Mon, May 21, 2012 at 02:32:43PM -0400, Stephen Gallagher wrote:
> > Strictly speaking, it would have been freed in the done: label. I
> > originally did that because I thought I was going to hang onto nval
> > longer. Since I clearly do not need to, I rearranged that so that it's
> > freed right after the conversion.
> > 
> > 
> > I made one additional change. I realized that we were overloading 'ret'
> > with both errno and ldap return values. I created a new variable 'lret'
> > to differentiate them and avoid polluting the values.
> 
> Ack

Pushed to master.

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to