Skip Menu |

Subject: Delete kadmin/passwd
Date: Wed, 13 Nov 2002 07:44:40 -0500 (EST)
From: hartmans@MIT.EDU (Sam Hartman)

People should use clients/kpasswd instead.

Deleting this may require changing tests depending on whether
kadmin/passwd is used by the tests.
Subject: delete kadmin v1 support
We should get rid of the v1 kadmin support.

Certainly get rid of the support in admin.h if we make it public, to prevent anyone from using it
in new programs. Ideally, kill the support in the library and server as well, if we aren't
supporting anything that needs it.

I think this would mean getting rid of the ancient password-changing service built into
kadmind, for which we haven't installed a client program in a long time. And we can finally kill
the client^H^H^H^H^H^Htest program we use to make sure the server still works (see ticket
1255), and a bunch of ugly code in the library that makes functions take different data
structures depending on the caller's macro definitions, all the ovsec_* wrapper functions, etc.
It'll also make creating an rpcgen input file for the kadmin protocol much easier.
Sent query to krbdev list 2008-08-04 (see
August/thread.html#6816) asking if anyone is using the ovsec_ or v1 kadm5_ APIs.
The only tests using kadmin/passwd are the kadmin/passwd unit tests. We could easily excise
that subtree, but some of the tests aren't duplicated elsewhere (changing to same password,
password in dictionary, character class requirements).
Subject: SVN Commit
Download (untitled) / with headers
text/plain 2.7KiB

The kadmin v1 API and the even older ovsec_kadm_* API were legacy when
kadmin was first incorporated in 1996, and compatibility with them is
no longer believed to be necessary.

The uninstalled kadmin/passwd has been removed (since it used the ovsec
API). The test suite has been updated to use the v2 API where
appropriate, and the parts specifically designed to test the old API
have been excised.
Commit By: ghudson
Revision: 22521
Changed Files:
U trunk/src/config/
U trunk/src/
U trunk/src/kadmin/
D trunk/src/kadmin/passwd/
U trunk/src/kadmin/server/
U trunk/src/kadmin/server/misc.h
U trunk/src/kadmin/server/ovsec_kadmd.c
D trunk/src/kadmin/server/server_glue_v1.c
U trunk/src/kadmin/server/server_stubs.c
U trunk/src/kadmin/testing/scripts/env-setup.shin
U trunk/src/kadmin/testing/scripts/init_db
U trunk/src/kadmin/testing/scripts/make-host-keytab.plin
U trunk/src/kadmin/testing/scripts/start_servers_local
U trunk/src/kadmin/testing/util/
U trunk/src/kadmin/testing/util/deps
U trunk/src/kadmin/testing/util/tcl_kadm5.c
U trunk/src/kadmin/testing/util/tcl_kadm5.h
A trunk/src/kadmin/testing/util/tcl_kadm5_syntax
D trunk/src/kadmin/testing/util/tcl_ovsec_kadm.c
D trunk/src/kadmin/testing/util/tcl_ovsec_kadm_syntax
U trunk/src/kadmin/testing/util/test.c
U trunk/src/lib/kadm5/
U trunk/src/lib/kadm5/admin.h
U trunk/src/lib/kadm5/admin_internal.h
U trunk/src/lib/kadm5/
U trunk/src/lib/kadm5/clnt/client_init.c
U trunk/src/lib/kadm5/clnt/client_principal.c
U trunk/src/lib/kadm5/clnt/clnt_policy.c
U trunk/src/lib/kadm5/clnt/libkadm5clnt.exports
U trunk/src/lib/kadm5/kadm_rpc_xdr.c
U trunk/src/lib/kadm5/misc_free.c
D trunk/src/lib/kadm5/ovsec_glue.c
U trunk/src/lib/kadm5/srv/libkadm5srv.exports
U trunk/src/lib/kadm5/srv/server_init.c
U trunk/src/lib/kadm5/srv/svr_misc_free.c
U trunk/src/lib/kadm5/srv/svr_policy.c
U trunk/src/lib/kadm5/srv/svr_principal.c
U trunk/src/lib/kadm5/unit-test/
D trunk/src/lib/kadm5/unit-test/
D trunk/src/lib/kadm5/unit-test/api.0/
D trunk/src/lib/kadm5/unit-test/api.1/lock.exp
U trunk/src/lib/kadm5/unit-test/config/unix.exp
U trunk/src/lib/kadm5/unit-test/destroy-test.c
U trunk/src/lib/kadm5/unit-test/handle-test.c
U trunk/src/lib/kadm5/unit-test/init-test.c
U trunk/src/lib/kadm5/unit-test/iter-test.c
U trunk/src/lib/kadm5/unit-test/lib/lib.t
U trunk/src/lib/kadm5/unit-test/lock-test.c
U trunk/src/lib/kadm5/unit-test/randkey-test.c
U trunk/src/lib/kadm5/unit-test/site.exp
U trunk/src/lib/rpc/unit-test/lib/helpers.exp
U trunk/src/lib/rpc/unit-test/
Subject: SVN Commit

In doc/Makefile, specify the new location of the kpasswd man page (the
old one was removed in r22521.
Commit By: ghudson
Revision: 22522
Changed Files:
U trunk/doc/Makefile
Subject: SVN Commit

Remove unused variables resulting from r22521, and also remove the
unused file svr_misc_free.c.
Commit By: ghudson
Revision: 22523
Changed Files:
U trunk/src/kadmin/server/server_stubs.c
U trunk/src/lib/kadm5/misc_free.c
D trunk/src/lib/kadm5/srv/svr_misc_free.c
U trunk/src/lib/kadm5/srv/svr_principal.c
Subject: SVN Commit

Remove bogus conditional from tcl_kadm5_randkey_principal(), which was
causing unparse_keyblocks() to get stack garbage as num_keys when
num_var was "null", thus overrunning the end of the array.
Commit By: tlyu
Revision: 22645
Changed Files:
U trunk/src/kadmin/testing/util/tcl_kadm5.c
Subject: SVN Commit

Update a kadm5 testing library function which was calling
kadm5_get_principal without a mask argment. This was causing many
lib/kadm5 tests to fail, but the failures weren't being recorded
properly, so "make check" was still exiting successfully.
Commit By: ghudson
Revision: 22646
Changed Files:
U trunk/src/lib/kadm5/unit-test/lib/lib.t