On Fri, Jan 16, 2015 at 09:28:30PM +0100, Lukas Slebodnik wrote:
> On (15/01/15 11:15), Sumit Bose wrote:
> >Hi,
> >
> >this patch should fix
https://fedorahosted.org/sssd/ticket/2557 . Please
> >see the commit message for details.
> >
> >bye,
> >Sumit
>
> realmd test (regression) passed.
> ldap + krb5 passed.
> ad passed.
>
> There are two tiny warnings from static analysers.
>
>
Thank you for the checks and the review, new version attached.
bye,
Sumit
From cb449c569fe18f0a5fc150de4c6724ab0a3146e5 Mon Sep 17 00:00:00
2001
From: Sumit Bose <sbose(a)redhat.com>
Date: Thu, 15 Jan 2015 10:38:33 +0100
Subject: [PATCH] krb5: fix entry order in MEMORY keytab
Since krb5_kt_add_entry() adds new entries at the beginning of a MEMORY
type keytab and not at the end a simple copy into a MEMORY type keytab
will revert the order of the keytab entries. Since e.g. the sssd_krb5
man page give hints about where to add entries into keytab files to help
SSSD to find a right entry we have to keep the order when coping a
keytab into a MEMORY type keytab. This patch fixes this by doing a
second copy to retain the original order.
Resolves
https://fedorahosted.org/sssd/ticket/2557
---