[Freeipa-devel] [PATCH] added PAM default configuration to confdb_init_db

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Sumit Bose wrote:
> see $subject.
> 
> bye,
> Sumit
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Freeipa-devel mailing list
> Freeipa-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/freeipa-devel

I overlooked this when I wrote that function in the first place, but
could you do a NULL check on all of the val[0] assignments that call
talloc_asprintf()? We should be handling out-of-memory properly. Passing
a NULL value of val[0] *will* cause a segfault in confdb_add_param().

- --
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iEYEARECAAYFAkmxGhcACgkQeiVVYja6o6Nh4ACdFcWzTZnC2HtLFZZVl8uMq2xF
+LoAn1SWz1DbYIWgET1Vluy8ZoLbX6uO
=nRen
-----END PGP SIGNATURE-----