set-change password breaks kpasswd

In some cases a null realm argument was passed into the function for
locating the kpasswd server.  This ended up causing segfaults in
kpasswd.  Fix to use the right realm.

ticket: new
Tags: pullup
Target_Version: 1.3

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

diff --git a/src/lib/krb5/os/ChangeLog b/src/lib/krb5/os/ChangeLog
index a53810545b1d2f51976f7fe9d751f1b55d6036b1..a17633ad7faf7777751d0790a2e99823eed1372d 100644 (file)
--- a/src/lib/krb5/os/ChangeLog
+++ b/src/lib/krb5/os/ChangeLog
@@ -1,3 +1,9 @@
+2003-04-28  Sam Hartman  <hartmans@mit.edu>
+
+       * changepw.c (krb5_change_set_password): Locate server in realm of
+       creds.server, not in realm of target principal because target
+       principal is null in the  changepw case.
+
 2003-04-28  Ken Raeburn  <raeburn@mit.edu>
 
        * init_os_ctx.c (krb5_os_init_context, krb5_os_free_context):

diff --git a/src/lib/krb5/os/changepw.c b/src/lib/krb5/os/changepw.c
index 0414848214bc74117f270a06f4ff285257a64f0d..f8ec224dba504430e299218a1557736a541d4e64 100644 (file)
--- a/src/lib/krb5/os/changepw.c
+++ b/src/lib/krb5/os/changepw.c
@@ -116,7 +116,7 @@ krb5_change_set_password(
          goto cleanup;
 
     if ((code = krb5_locate_kpasswd(context,
-                                    krb5_princ_realm(context, set_password_for),
+                                    krb5_princ_realm(context, creds->server),
                                    &al)))
         goto cleanup;