* api.0/init.exp, api.1/lock.exp, api.2/init.exp: use $K5ROOT
authorBarry Jaspan <bjaspan@mit.edu>
Thu, 26 Sep 1996 21:46:40 +0000 (21:46 +0000)
committerBarry Jaspan <bjaspan@mit.edu>
Thu, 26 Sep 1996 21:46:40 +0000 (21:46 +0000)
  instead of /krb5

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@9120 dc483132-0cff-0310-8789-dd5450dbe970

src/lib/kadm5/ChangeLog
src/lib/kadm5/alt_prof.c
src/lib/kadm5/unit-test/api.0/init.exp
src/lib/kadm5/unit-test/api.1/lock.exp
src/lib/kadm5/unit-test/api.2/init.exp

index 28bc24f3be5ddc86b3345169ebaea7b225d94b10..411bc75f654f594070d8b5b756c5113bbae88dcf 100644 (file)
@@ -1,3 +1,8 @@
+Thu Sep 26 17:45:18 1996  Barry Jaspan  <bjaspan@mit.edu>
+
+       * alt_prof.c: don't allow admin_dbname or admin_lockfile to be
+       independetly configured
+
 Fri Sep 20 16:52:07 1996  Barry Jaspan  <bjaspan@mit.edu>
 
        * configure.in: add unit-test directory
index 71b25978684e2ba3723bda07919e31bf39e8b823..374e22706e017cfe0ebf17430fc156282bf38245 100644 (file)
@@ -364,35 +364,9 @@ krb5_error_code kadm5_get_config_params(context, kdcprofile, kdcenv,
     }
 
     /*
-     * Get the value for the admin (policy) database and lockfile.
-     * The logic here is slightly tricky.  DBNAME, ADBNAME, and
-     * ADB_LOCKFILE are dependent on the earlier items in the
-     * sequence.  If an earlier item was specified via the input
-     * parameters, that value overrides the variables in the config
-     * file and causes the later item to be set to ".kadm5" or
-     * ".lock", respectively.  However, if no earlier item was
-     * specified, the variables in the config file are used, and the
-     * ".kadm5" and ".lock" suffixes are only added as a no-variable
-     * default.
-     *
-     * Read the spec.
+     * admin database name and lockfile are now always derived from dbname
      */
-    hierarchy[2] = "admin_database_name";
-    if (params_in->mask & KADM5_CONFIG_ADBNAME) {
-        params.admin_dbname = strdup(params_in->admin_dbname);
-        if (params.admin_dbname)
-             params.mask |= KADM5_CONFIG_ADBNAME;
-    } else if (params_in->mask & KADM5_CONFIG_DBNAME) {
-        params.admin_dbname = (char *) malloc(strlen(params.dbname) + 7);
-        if (params.admin_dbname) {
-             sprintf(params.admin_dbname, "%s.kadm5", params.dbname);
-             params.mask |= KADM5_CONFIG_ADBNAME;
-        }
-    } else if (aprofile &&
-              !krb5_aprof_get_string(aprofile, hierarchy, TRUE, &svalue)) {
-        params.admin_dbname = svalue;
-        params.mask |= KADM5_CONFIG_ADBNAME;
-    } else if (params.mask & KADM5_CONFIG_DBNAME) {
+    if (params.mask & KADM5_CONFIG_DBNAME) {
         params.admin_dbname = (char *) malloc(strlen(params.dbname) + 7);
         if (params.admin_dbname) {
              sprintf(params.admin_dbname, "%s.kadm5", params.dbname);
@@ -400,27 +374,7 @@ krb5_error_code kadm5_get_config_params(context, kdcprofile, kdcenv,
         }
     }
 
-    /* Get the value for the admin (policy) database lock file*/
-    hierarchy[2] = "admin_database_lockfile";
-    if (params_in->mask & KADM5_CONFIG_ADB_LOCKFILE) {
-        params.admin_lockfile = strdup(params_in->admin_lockfile);
-        if (params.admin_lockfile)
-             params.mask |= KADM5_CONFIG_ADB_LOCKFILE;
-    } else if ((params_in->mask & KADM5_CONFIG_ADBNAME) ||
-              (params_in->mask & KADM5_CONFIG_DBNAME)) {
-        /* if DBNAME is set but ADBNAME is not, then admin_database
-         * will already have been set above */
-        params.admin_lockfile = (char *) malloc(strlen(params.admin_dbname)
-                                                + 6); 
-        if (params.admin_lockfile) {
-             sprintf(params.admin_lockfile, "%s.lock", params.admin_dbname);
-             params.mask |= KADM5_CONFIG_ADB_LOCKFILE;
-        }
-    } else if (aprofile &&
-              !krb5_aprof_get_string(aprofile, hierarchy, TRUE, &svalue)) {
-        params.mask |= KADM5_CONFIG_ADB_LOCKFILE;
-        params.admin_lockfile = svalue;
-    } else if (params.mask & KADM5_CONFIG_ADBNAME) {
+    if (params.mask & KADM5_CONFIG_ADBNAME) {
         params.admin_lockfile = (char *) malloc(strlen(params.admin_dbname)
                                                 + 6);
         if (params.admin_lockfile) {
index 08fed8b18444f75d1b20b98c3322e413889b21ee..eec097ea1506da63c37d7b4b91e3e1b2e1b23471 100644 (file)
@@ -716,9 +716,10 @@ if {$RPC} {
     test45_46 ovsec_adm/changepw
 
     # re-extract the keytab so it is right
-    exec rm /krb5/ovsec_adm.srvtab
+    exec rm $env(K5ROOT)/ovsec_adm.srvtab
     exec $env(MAKE_KEYTAB) -princ ovsec_adm/admin -princ ovsec_adm/changepw \
-           -princ kadmin/admin -princ kadmin/changepw /krb5/ovsec_adm.srvtab
+           -princ kadmin/admin -princ kadmin/changepw \
+           $env(K5ROOT)/ovsec_adm.srvtab
 }
 
 return ""
index a646b328751381f40621a863602a7ef373aa9dbf..f9a30202716164447a262a7a781bf2e8fbbd0c80 100644 (file)
@@ -12,7 +12,7 @@ if { $RPC } {
 }
 
 set locktest $LOCKTEST
-set lockfile "/krb5/kdb5.kadm5.lock"
+set lockfile $env(K5ROOT)/kdb5.kadm5.lock
 
 # The lock tests use the program lock-test in the unit test
 # directory.  The basic idea is that lock-test can be told to acquire
index 9707d3bf31bb930aaf1135b033893a1956a54d06..79efcee150a66778e58aeae8f8bcade07e4bfca8 100644 (file)
@@ -720,9 +720,10 @@ if {$RPC} {
     test45_46 ovsec_adm/changepw
 
     # re-extract the keytab so it is right
-    exec rm /krb5/ovsec_adm.srvtab
+    exec rm $env(K5ROOT)/ovsec_adm.srvtab
     exec $env(MAKE_KEYTAB) -princ ovsec_adm/admin -princ ovsec_adm/changepw \
-           -princ kadmin/admin -princ kadmin/changepw /krb5/ovsec_adm.srvtab
+           -princ kadmin/admin -princ kadmin/changepw \
+           $env(K5ROOT)/ovsec_adm.srvtab
 }
 
 return ""