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
ACK.
Looks good and passes the CI tests:
http://sssd-ci.duckdns.org/logs/commit/95/4ab5bda3d3e4e36a77507e241371a6b...
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 :-)