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

Jakub Hrozek jhrozek at redhat.com
Fri Jun 14 08:03:28 UTC 2013


On Thu, Jun 13, 2013 at 12:20:31PM -0400, Pavel Brezina wrote:
> 
> 
> ----- 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?

Just the hardcoded path.



More information about the sssd-devel mailing list