Skip Menu |
 

History Show all quoted text — Show full headers
# Mon Oct 07 15:21:50 2002 Ben Cox <cox-work@djehuti.com> - Ticket created
Subject: kadmind patch
From: Ben Cox <cox-work@djehuti.com>
To: krb5-bugs@mit.edu
Date: 07 Oct 2002 15:21:45 -0400
Download (untitled) / with headers
text/plain 866B
Hello,

The attached unified diff against the krb5-1.2.6 source tree fixes a bug
in kadmin where a principal changing his own key cannot specify a list
of supported enctypes (while principals changing other principals' keys
can do so).

To see the bug, configure more than one supported enctype in the KDC,
and create a test principal, and then run "kadmin -p <testprinc>". In
kadmin, try "cpw [-randkey] -e des-cbc-crc:normal <testprinc>". Then
"getprinc <testprinc>" shows that the list was not honored.

This patch fixes the problem my extending the server-side
{chpass,randkey}_principal_wrapper functions to take the keepold,
n_ks_tuple and ks_tuple arguments common to the
kadm5_{chpass,randkey}_principal_3 functions.

The attached patch has been tested with MIT Kerberos 5 version 1.2.6 on
Linux RedHat 7.3, but should pose no portability issues.

-- Ben
Download kadmin-diff.patch
text/plain 4.7KiB

Message body is not shown because sender requested not to inline it.

# Tue Oct 08 11:53:24 2002 tlyu (Taylor Yu) - Taken
# Tue Oct 08 16:53:32 2002 tlyu (Taylor Yu) - Status changed from 'new' to 'resolved'
# Tue Oct 08 16:53:32 2002 tlyu (Taylor Yu) - Version_reported 1.2.6 added
# Tue Oct 08 16:53:32 2002 tlyu (Taylor Yu) - Target_Version 1.3 added
# Tue Oct 08 16:53:33 2002 tlyu (Taylor Yu) - Subject changed from 'kadmind patch' to 'principal can't specify ks_tuples changing own passwd'
# Tue Oct 08 16:53:33 2002 tlyu (Taylor Yu) - Correspondence added
From: tlyu@mit.edu
Subject: CVS Commit
Download (untitled) / with headers
text/plain 735B
Thanks, the patch has been applied and will appear in a future release.

* misc.c (chpass_principal_wrapper_3): Renamed from
chpass_principal_wrapper; calls chpass_principal_3 now.
(randkey_principal_wrapper_3): Renamed from
randkey_principal_wrapper; calls randkey_principal_3 now. Patch
from Ben Cox.

* server_stubs.c (chpass_principal_1_svc)
(chpass_principal3_1_svc): Call chpass_principal_wrapper_3.
(chrand_principal_1_svc, chrand_principal3_1_svc): Call
randkey_principal_wrapper_3. Patch from Ben Cox.


To generate a diff of this commit:



cvs diff -r5.84 -r5.85 krb5/src/kadmin/server/ChangeLog
cvs diff -r1.13 -r1.14 krb5/src/kadmin/server/misc.c
cvs diff -r1.44 -r1.45 krb5/src/kadmin/server/server_stubs.c
# Thu Oct 31 18:34:50 2002 tlyu (Taylor Yu) - Target_Version 1.3 changed to 1.2.7
# Thu Oct 31 18:34:50 2002 tlyu (Taylor Yu) - Correspondence added
From: tlyu@mit.edu
Subject: CVS Commit
Download (untitled) / with headers
text/plain 348B
pull up Ben Cox's patch to allow a client to specify ks_tuples to
password changing functions


To generate a diff of this commit:



cvs diff -r5.67.2.2.2.1 -r5.67.2.2.2.2
krb5/src/kadmin/server/ChangeLog
cvs diff -r1.10 -r1.10.22.1 krb5/src/kadmin/server/misc.c
cvs diff -r1.40.2.1.2.1 -r1.40.2.1.2.2
krb5/src/kadmin/server/server_stubs.c
# Fri Nov 15 17:20:17 2002 tlyu (Taylor Yu) - Version_Fixed 1.2.7 added