On 01/27/2016 04:04 PM, Tomas Babej wrote: > > > On 01/06/2016 12:15 PM, Sumit Bose wrote: >> Hi, >> >> this patch fixes and issue found by Simo when he called >> get_authz_data_types() with the second argument being NULL. >> This function determines which type of authorization data should be >> added to the Kerberos ticket. There are global default and it is >> possible to configure this per service as well. The second argument is >> the data base entry of a service. If no service is given it makes sens >> to return the global defaults and most parts of get_authz_data_types() >> handle this case well and this patch fixes the remain issue and adds a >> test for this as well. >> >> Please note that currently get_authz_data_types() is used in a code path >> where the service entry is expected to be not NULL and it turned out >> that in Simo's case it will be non-NULL as well. Nevertheless the patch >> makes the code more robust and makes the future use of >> get_authz_data_types() more safe. >> >> bye, >> Sumit >> >> >> > > ACK, thanks. >
It is worth noting I added a commit message for the patch, based on the Sumit's summary here. Pushed to master: 45b0148fcce3fded5cea52b6fadd50114358ba25 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code