[SSSD] [PATCH] Fix krbcc dir creation issue with MIT krb5 1.11

Jakub Hrozek jhrozek at redhat.com
Mon Apr 15 09:44:29 UTC 2013


On Mon, Apr 15, 2013 at 10:56:49AM +0200, Lukas Slebodnik wrote:
> On (13/04/13 00:31), Lukas Slebodnik wrote:
> >On (12/04/13 18:06), Jakub Hrozek wrote:
> >>On Fri, Apr 12, 2013 at 05:31:33PM +0200, Jakub Hrozek wrote:
> >>> On Thu, Apr 11, 2013 at 12:06:22PM +0200, Lukas Slebodnik wrote:
> >>> > On (11/04/13 10:29), Jakub Hrozek wrote:
> >>> > >On Thu, Apr 11, 2013 at 10:28:01AM +0200, Jakub Hrozek wrote:
> >>> > >> Just one nitpick, can you change this block:
> >>> > >> 
> >>> > >> +    primary_file = talloc_strdup_append(discard_const_p(char, dir),
> >>> > >> +                                        "/primary");
> >>> > >> +    if (!primary_file) {
> >>> > >> +        DEBUG(SSSDBG_CRIT_FAILURE, ("talloc_strdup_append failed.\n"));
> >>> > >> +        ret = ENOMEM;
> >>> > >> +        goto done;
> >>> > >> +    }
> >>> > >> 
> >>> > >> Change dir to be "char *" not "const char*" and use
> >>> > >> talloc_asprintf(tmp_ctx, "%s/%s", dir);
> >>> > >
> >>> > >talloc_asprintf(tmp_ctx, "%s/primary", dir);
> >>> > >
> >>> > >sorry.
> >>> > 
> >>> > Patch attached.
> >>> > 
> >>> > LS
> >>> 
> >>> Ack, works fine. I will just change one thing before pushing, no need to
> >>> re-send:
> >>> 
> >>> >      ret = cc_residual_is_used(uid, dir, SSS_KRB5_TYPE_DIR, &active);
> >>> > -    talloc_free(tmp);
> >>> >      if (ret != EOK) {
> >>> > -        DEBUG(SSSDBG_OP_FAILURE, ("Could not check if ccache is active. "
> >>> > -                                  "Will create a new one.\n"));
> >>> > +        if (ret != ENOENT) {
> >>> > +            DEBUG(SSSDBG_OP_FAILURE,
> >>> > +                  ("Could not check if ccache is active.\n"));
> >>> > +        }
> >>> 
> >>> I will just change the debug level here to something more verbose. I
> >>> think we should let the user know why the login failed.
> >>
> >>Pushed to master.
> >>
> >>Lukas, can you also send a version that applies cleanly on the sssd-1-9
> >>branch? I think we should fix this bug in 1.9 too.
> >
> >Attaching patch for branch sssd-1-9.
> >I did not test patch on RHEL 6, I have to create VM.
> >
> >LS
> 
> Patch for sssd-1-9 works fine on RHEL6,
> but branch sssd-1-9 does not compile on with krb5-libs 1.11 (fedora 19).
> 
> You should also backport patch 4e78fab6a1b2e9653a7959cbdb7d54bb750041d0
> "krb5: include backwards compatible declaration of krb5_trace_info"
> from master to sssd-1-9.

Yes, luckily that patch applies fine without backporting so I will just
push it.



More information about the sssd-devel mailing list