From: Sam Hartman <hartmans@mit.edu>
Date: Mon, 28 Apr 2003 21:38:02 +0000 (+0000)
Subject: set-change password breaks kpasswd
X-Git-Tag: krb5-1.4-beta1~992
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=6eb88d3b13160c110d92079ef2324161502b9ddf;p=krb5.git

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 a53810545..a17633ad7 100644
--- 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 041484821..f8ec224db 100644
--- 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;