Attachment 585245 Details for Bug 822350 – Fix for the failure to change the password

[patch] Fix for the failure to change the password

0001-Fix-migration-code-password-setting.patch (text/plain), 1.56 KB, created by Simo Sorce on 2012-05-17 14:42:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Simo Sorce

Created: 2012-05-17 14:42:54 UTC

Size: 1.56 KB

patch

obsolete

>From 2b7ffadfe6998d81f5f3c8f89715332b7c9bc466 Mon Sep 17 00:00:00 2001
>From: Simo Sorce <ssorce@redhat.com>
>Date: Thu, 17 May 2012 10:33:43 -0400
>Subject: [PATCH] Fix migration code password setting.
>
>When we set a password we also need to make sure krbExtraData is set.
>If not kadmin will later complain that the object is corrupted at password
>change time.
>---
> .../ipa-pwd-extop/ipapwd_prepost.c                 |    7 +++++++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
>diff --git a/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_prepost.c b/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_prepost.c
>index 410c536a5307219a75d5e8db7ab1ce77f4968a39..879f7fa620bf788170c2c6baff72315a24ea7545 100644
>--- a/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_prepost.c
>+++ b/daemons/ipa-slapi-plugins/ipa-pwd-extop/ipapwd_prepost.c
>@@ -995,6 +995,7 @@ static int ipapwd_pre_bind(Slapi_PBlock *pb)
>     Slapi_Value *objectclass;
>     int method; /* authentication method */
>     int ret = 0;
>+    char *principal = NULL;
> 
>     LOG_TRACE("=>\n");
> 
>@@ -1135,9 +1136,15 @@ static int ipapwd_pre_bind(Slapi_PBlock *pb)
>         goto done;
>     }
> 
>+    principal = slapi_entry_attr_get_charptr(entry, "krbPrincipalName");
>+    /* we need to make sure the ExtraData is set, otherwise kadmin
>+     * will not like the object */
>+    ipapwd_set_extradata(pwdata.dn, principal, pwdata.timeNow);
>+
>     LOG("kerberos key generated for user entry: %s\n", dn);
> 
> done:
>+    slapi_ch_free_string(&principal);
>     slapi_ch_free_string(&expire);
>     if (entry)
>         slapi_entry_free(entry);
>-- 
>1.7.7.6
>

Actions: View | Diff

Attachments on bug 822350: 585245