[SSSD] [PATCH] subdomains: touch krb5.conf when creating new domain-realm mappings

Pavel Brezina pbrezina at redhat.com
Thu Jun 13 16:20:31 UTC 2013



----- Original Message -----
> From: "Jakub Hrozek" <jhrozek at redhat.com>
> To: sssd-devel at lists.fedorahosted.org
> Sent: Thursday, June 13, 2013 5:14:13 PM
> Subject: Re: [SSSD] [PATCH] subdomains: touch krb5.conf when creating new domain-realm mappings
> 
> On Thu, Jun 13, 2013 at 03:48:37PM +0200, Pavel Březina wrote:
> > https://fedorahosted.org/sssd/ticket/1815
> 
> > +errno_t sss_krb5_touch_config()
> > +{
> > +    const char *config = NULL;
> > +    errno_t ret;
> > +
> > +    config = getenv("KRB5_CONFIG");
> > +    if (config == NULL) {
> > +        config = "/etc/krb5.conf";
> > +    }
> 
> I had a discussion with ab on IRC and the consensus was that we really
> should make the location configurable. So this code works fine, we just
> also need a configure-time macro to set the krb5.conf location. The
> macro should default to @sysconfdir@/krb5.conf

OK. Just to be clear, I should only replace "/etc/krb5.conf" with value from configure or I should remove getenv("KRB5_CONFIG") also?



More information about the sssd-devel mailing list