This modification causes ceod to add the Kerberos principal.
authorMarc Burns <m4burns@csclub.uwaterloo.ca>
Fri, 16 Mar 2012 19:15:59 +0000 (15:15 -0400)
committerMarc Burns <m4burns@csclub.uwaterloo.ca>
Fri, 16 Mar 2012 19:15:59 +0000 (15:15 -0400)
It returns an error code to ceoc, which I will now fix.

src/kadm.c

index 44af913..74b46ac 100644 (file)
@@ -36,22 +36,42 @@ void ceo_kadm_cleanup() {
 
 int ceo_add_princ(char *user, char *password) {
     krb5_error_code retval;
 
 int ceo_add_princ(char *user, char *password) {
     krb5_error_code retval;
-    krb5_principal princ;
-    memset((void *) &princ, 0, sizeof(princ));
 
     debug("kadmin: adding principal %s", user);
 
 
     debug("kadmin: adding principal %s", user);
 
-    if ((retval = krb5_parse_name(context, user, &princ))) {
-        com_err(prog, retval, "while parsing principal name");
+    // Added March 2012: Change behavior of ceod to add the kerberos principal.
+    kadm5_policy_ent_rec defpol;
+    kadm5_principal_ent_rec princ;
+
+    memset((void*) &princ, 0, sizeof(princ));
+
+    if ((retval = kadm5_get_policy(handle, "default", &defpol))) {
+        com_err(prog, retval, "while retrieving default policy");
         return retval;
     }
         return retval;
     }
+    kadm5_free_policy_ent(handle, &defpol);
 
 
-    if ((retval = kadm5_chpass_principal(handle, princ, password))) {
-        com_err(prog, retval, "while creating principal");
+    princ.policy = "default";
+
+    if ((retval = krb5_parse_name(context, user, &princ.principal))) {
+        com_err(prog, retval, "while parsing user name");
         return retval;
     }
 
         return retval;
     }
 
-    krb5_free_principal(context, princ);
+    long flags = KADM5_POLICY | KADM5_PRINCIPAL;
+    if ((retval = kadm5_create_principal(handle, &princ, flags, password))) {
+        if(retval == KADM5_DUP) {
+            if ((retval = kadm5_chpass_principal(handle, princ.principal, password))) {
+                com_err(prog, retval, "while setting principal password");
+                return retval;
+            }
+        } else {
+            com_err(prog, retval, "while creating principal");
+            return retval;
+        }
+    }
+
+    krb5_free_principal(context, princ.principal);
     return 0;
 }
 
     return 0;
 }