On (19/01/15 09:23), Sumit Bose wrote:
>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
>---
ACK++
http://sssd-ci.duckdns.org/logs/job/6/19/summary.html
master: 576ad637181b80d39a4e136c9afbf34c57f76156
sssd-1-12: 24df1487413d13248dcc70d2548a763930da4c65