Pull up r23492 from branches/anonymous
authorTom Yu <tlyu@mit.edu>
Tue, 12 Jan 2010 23:04:32 +0000 (23:04 +0000)
committerTom Yu <tlyu@mit.edu>
Tue, 12 Jan 2010 23:04:32 +0000 (23:04 +0000)
commit923455eea32b35d5ad32c527d3eb5f63f2499827
treea403addfd702c20fde0e209db29aec70245e1c50
parent084ec81befc937eac81157dd3995092e1de7e566
Pull up r23492 from branches/anonymous

 ------------------------------------------------------------------------
 r23492 | hartmans | 2009-12-23 16:09:50 -0500 (Wed, 23 Dec 2009) | 17 lines

 Subject: ad-initial-verified-cas logic broken
 ticket: 6587
 status: open

 In the initial pkinit implementation, the server plugin generates an
 incorrect encoding for ad-initial-verified-cas.  In particular, it
 assumes that ad-if-relevant takes a single authorization data element
 not a sequence of authorization data elements.  Nothing looked at the
 authorization data in 1.6.3 so this was not noticed.  However in 1.7,
 the FAST implementation looks for authorization data.  In 1.8 several
 more parts of the KDC examine authorization data.  The net result is
 that the KDC fails to process the TGT it issues.

 However on top of this bug, there is a spec problem.  For many of its
 intended uses, ad-initial-verified-cas needs to be integrity
 protected by the KDC in order to prevent a client from injecting it.
 So, it should be contained in kdc-issued not ad-if-relevant.

 For now we're simply removing the generation of this AD element until
 the spec is clarified.

------------------------------------------------------------------------

ticket: 6587
version_fixed: 1.7.1
status: resolved

git-svn-id: svn://anonsvn.mit.edu/krb5/branches/krb5-1-7@23654 dc483132-0cff-0310-8789-dd5450dbe970
src/plugins/preauth/pkinit/pkinit_srv.c