svn rev #23807: branches/krb5-1-8/src/lib/kadm5/srv/

[tlyu@MIT.EDU]

http://src.mit.edu/fisheye/changelog/krb5/?cs=23807
Commit By: tlyu
Log Message:
ticket: 6668
version_fixed: 1.8.1
status: resolved

pull up r23749 from trunk

 ------------------------------------------------------------------------
 r23749 | ghudson | 2010-02-24 13:57:08 -0500 (Wed, 24 Feb 2010) | 9 lines

 ticket: 6668
 subject: Two problems in kadm5_get_principal mask handling
 target_version: 1.8
 tags: pullup

 KADM5_MOD_NAME was being applied to entry->principal instead of
 entry->mod_name.  KADM5_MKVNO was not being applied to entry->mkvno.
 Patch from Marcus Watts <mdw at umich.edu>.


Changed Files:
U   branches/krb5-1-8/src/lib/kadm5/srv/svr_principal.c
Modified: branches/krb5-1-8/src/lib/kadm5/srv/svr_principal.c
===================================================================
--- branches/krb5-1-8/src/lib/kadm5/srv/svr_principal.c	2010-03-15 23:50:40 UTC (rev 23806)
+++ branches/krb5-1-8/src/lib/kadm5/srv/svr_principal.c	2010-03-15 23:50:46 UTC (rev 23807)
@@ -858,8 +858,8 @@
         if (! (mask & KADM5_MOD_TIME))
             entry->mod_date = 0;
         if (! (mask & KADM5_MOD_NAME)) {
-            krb5_free_principal(handle->context, entry->principal);
-            entry->principal = NULL;
+            krb5_free_principal(handle->context, entry->mod_name);
+            entry->mod_name = NULL;
         }
     }
 
@@ -871,10 +871,12 @@
             if (kdb.key_data[i].key_data_kvno > entry->kvno)
                 entry->kvno = kdb.key_data[i].key_data_kvno;
 
-    ret = krb5_dbe_get_mkvno(handle->context, &kdb, master_keylist,
-                             &entry->mkvno);
-    if (ret)
-        goto done;
+    if (mask & KADM5_MKVNO) {
+        ret = krb5_dbe_get_mkvno(handle->context, &kdb, master_keylist,
+                                 &entry->mkvno);
+        if (ret)
+            goto done;
+    }
 
     if (mask & KADM5_MAX_RLIFE)
         entry->max_renewable_life = kdb.max_renewable_life;