[26244] in CVS-changelog-for-Kerberos-V5
svn rev #25584: trunk/src/lib/krb5/krb/
daemon@ATHENA.MIT.EDU (ghudson@mit.edu)
Mon Dec 12 19:54:15 2011
Date: Mon, 12 Dec 2011 19:53:56 -0500
From: ghudson@mit.edu
Message-Id: <201112130053.pBD0ru1s029709@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu
http://src.mit.edu/fisheye/changelog/krb5/?cs=25584
Commit By: ghudson
Log Message:
ticket: 7051
subject: krb5_server_decrypt_ticket_keytab wrongly succeeds
If krb5_server_decrypt_ticket_keytab doesn't find a key of the
appropriate enctype in an iterable keytab, it returns 0 (without
decrypting the ticket) due to a misplaced initialization of retval.
This bug causes kinit -k to claim "keytab entry valid" when it
shouldn't. Reported by mark@mproehl.net.
Changed Files:
U trunk/src/lib/krb5/krb/srv_dec_tkt.c
Modified: trunk/src/lib/krb5/krb/srv_dec_tkt.c
===================================================================
--- trunk/src/lib/krb5/krb/srv_dec_tkt.c 2011-12-12 20:47:14 UTC (rev 25583)
+++ trunk/src/lib/krb5/krb/srv_dec_tkt.c 2011-12-13 00:53:56 UTC (rev 25584)
@@ -79,8 +79,6 @@
krb5_error_code retval;
krb5_keytab_entry ktent;
- retval = KRB5_KT_NOTFOUND;
-
if (keytab->ops->start_seq_get == NULL) {
retval = krb5_kt_get_entry(context, keytab,
ticket->server,
@@ -99,6 +97,7 @@
if (retval != 0)
goto map_error;
+ retval = KRB5_KT_NOTFOUND;
while ((code = krb5_kt_next_entry(context, keytab,
&ktent, &cursor)) == 0) {
if (ktent.key.enctype != ticket->enc_part.enctype)
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5
