On Wed, Jan 29, 2014 at 05:49:03PM +0100, Lukas Slebodnik wrote: > On (29/01/14 16:03), Sumit Bose wrote: > >On Wed, Jan 29, 2014 at 02:53:15PM +0100, Jakub Hrozek wrote: > >> On Wed, Jan 29, 2014 at 02:14:38PM +0100, Jakub Hrozek wrote: > >> > On Wed, Jan 29, 2014 at 01:33:52PM +0100, Sumit Bose wrote: > >> > > Code looks good and my tests passed, so ACK. > >> > > > >> > > bye, > >> > > Sumit > >> > > >> > Thank you for the review, pushed to master. There are some conflicts in > >> > the 1.11 branch, so I'll send patches separately. > >> > >> One conflict occured because 1.11 didn't have the patch > >> 1101a3da9037f69a556935f2706b844accc468de from master. Because the > >> conflict resulution would just do the same as that patch, I simply > >> cherry-picked it from master. > >> > >> With that extra patch applied, the only differences are that sysdb calls > >> also pass on the sysdb context in 1.11. All patches are attached for > >> posterity, but also interdiffs in the two patches that changed, for > >> easier review. > > > >Patches apply and pass some basic tests. ACK. > > > >Btw, I've seen two compiler warning when building 1.11 which seems to be > >fixed in master. Do you want the fixes for 1.11 as well? > > > >bye, > >Sumit > > Do you mean this warning? > https://lists.fedorahosted.org/pipermail/sssd-devel/2013-October/016963.html
yes, and a missing check of the return value of asprintf(). See my other mail for details. bye, Sumit > > LS > _______________________________________________ > sssd-devel mailing list > sssd-devel@lists.fedorahosted.org > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel