On Thu, Dec 11, 2014 at 01:30:21PM +0100, Sumit Bose wrote:
> On Thu, Dec 11, 2014 at 01:27:19PM +0100, Sumit Bose wrote:
> > Hi,
> > this is just a test for one of the new view/override related sysdb
> > calls.
> > bye,
> > Sumit
> and now with patch ....
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel(a)lists.fedorahosted.org
> > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
Looks good and passes the CI tests:
I also think using talloc_get_type_abort() in unit tests is a good idea.
In non-test code, it might be better to check the pointer validity and
fail, but I don't think we do that on most places :-)