Mark Phalan wrote: > Need a code review for the following: > > 6658621 Configuration checks for kerberos daemons should be done by the > daemons themselves > 6658631 error messages in kerberos deamons need cleanup > 6664832 various memleaks in krb libs > 6658627 kpropd should use its executable name and not the full path when > logging error messages > 6658624 Missing error strings for new kerberos DB error types > > The following related CRs will be closed once the above is putback: > > 6522924 Failure to start svc:/network/security/krb5kdc should be more > verbose > 6646459 'db_exists' and 'kadm5_acl_configed' tests broken in > svc-kdc/svc-kdc.master > 6623803 db_exists() in usr/src/cmd/svc/shell/krb_include.sh needs to be > updated to deal with LDAP > > webrev can be found here: > http://cr.opensolaris.org/~mbp/daemon_error_messages/ > ok, attached is my review...glenn
-------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: code-rev-mark-errmsgs.txt URL: <http://mail.opensolaris.org/pipermail/kerberos-discuss/attachments/20080303/89915651/attachment.txt>
