* ldap_principal.c (attributes_set): Swap first two elements

Also add comments indicating that this array and the KDB_*_ATTR macros
need to be in sync.

ticket: 5260

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19031 dc483132-0cff-0310-8789-dd5450dbe970

diff --git a/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c b/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c
index 83ba05d4ac909768b59d88a41cb7089a32e0ef1c..77256482eed2c26e6da82788069512d31bacc9b1 100644 (file)
--- a/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c
+++ b/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.c
@@ -65,8 +65,9 @@ char     *principal_attributes[] = { "krbprincipalname",
                                     "krbObjectReferences",
                                     NULL };
 
-static char *attributes_set[] = { "krbmaxrenewableage",
-                                 "krbmaxticketlife",
+/* Must match KDB_*_ATTR macros in ldap_principal.h.  */
+static char *attributes_set[] = { "krbmaxticketlife",
+                                 "krbmaxrenewableage",
                                  "krbticketflags",
                                  "krbprincipalexpiration",
                                  "krbticketpolicyreference",

diff --git a/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.h b/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.h
index e932e253e7675b85136ba33065e6b78d103bbb9a..ca8c80681c9c1fe946e89f146abf92cf2484e3ad 100644 (file)
--- a/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.h
+++ b/src/plugins/kdb/ldap/libkdb_ldap/ldap_principal.h
@@ -66,6 +66,7 @@
 /* these will be consumed only by krb5_ldap_delete_principal*/
 /* these will be set by krb5_ldap_get_principal and fed into the tl_data */
 
+/* See also attributes_set[] in ldap_principal.c.  */
 #define KDB_MAX_LIFE_ATTR                    0x000001
 #define KDB_MAX_RLIFE_ATTR                   0x000002
 #define KDB_TKT_FLAGS_ATTR                   0x000004