* kdb5.c (kdb_get_conf_section): If the default realm is null, return null.
authorKen Raeburn <raeburn@mit.edu>
Tue, 4 Oct 2005 20:27:33 +0000 (20:27 +0000)
committerKen Raeburn <raeburn@mit.edu>
Tue, 4 Oct 2005 20:27:33 +0000 (20:27 +0000)
(krb5_db_open): Don't pass null pointer to sprintf if default_realm is null.

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

src/lib/kdb/ChangeLog
src/lib/kdb/kdb5.c

index cf4127b1d14f927bacef4ed7b809979f248f9222..7107c50f2a01e6f7565e9da7002f0482be55b3db 100644 (file)
@@ -1,3 +1,10 @@
+2005-10-04  Ken Raeburn  <raeburn@mit.edu>
+
+       * kdb5.c (kdb_get_conf_section): If the default realm is null,
+       return null.
+       (krb5_db_open): Don't pass null pointer to sprintf if
+       default_realm is null.
+
 2005-09-22  Ken Raeburn  <raeburn@mit.edu>
 
        * kdb_db2: Directory moved to modules/kdb/db2.
index 82bcf8f32f053dec7dbcffb1c6c7b6b1813596cd..67f03e02f7e2faa758a51f5addc1d87dfe6d0c2d 100644 (file)
@@ -171,7 +171,10 @@ kdb_get_conf_section(krb5_context kcontext)
     char   *result = NULL;
     char   *value = NULL;
 
-    /* profile has to be initialized. If profile is not initialized, expect nothing less than a crash */
+    if (kcontext->default_realm == NULL)
+       return NULL;
+    /* The profile has to have been initialized.  If the profile was
+       not initialized, expect nothing less than a crash.  */
     status = profile_get_string(kcontext->profile,
                                /* realms */
                                KDB_REALM_SECTION,
@@ -638,7 +641,7 @@ krb5_db_open(krb5_context kcontext, char **db_args, int mode)
     if (section == NULL) {
        sprintf(buf,
                "unable to determine configuration section for realm %s\n",
-               kcontext->default_realm);
+               kcontext->default_realm ? kcontext->default_realm : "[UNSET]");
        status = -1;
        krb5_set_err(kcontext, krb5_err_have_str, status, buf);
        goto clean_n_exit;