Author: jra Date: 2004-10-30 00:34:50 +0000 (Sat, 30 Oct 2004) New Revision: 3378
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=3378 Log: More merging of kerberos keytab and salting fixes from Nalin Dahyabhai <[EMAIL PROTECTED]> (bugid #1717). Jeremy. Modified: trunk/source/libads/kerberos.c trunk/source/libads/kerberos_keytab.c trunk/source/libsmb/clikrb5.c Changeset: Modified: trunk/source/libads/kerberos.c =================================================================== --- trunk/source/libads/kerberos.c 2004-10-29 22:38:10 UTC (rev 3377) +++ trunk/source/libads/kerberos.c 2004-10-30 00:34:50 UTC (rev 3378) @@ -608,7 +608,18 @@ char *service_principal) { int i; + BOOL free_ccache = False; + if (ccache == NULL) { + krb5_error_code ret; + if ((ret = krb5_cc_resolve(context, LIBADS_CCACHE_NAME, &ccache)) != 0) { + DEBUG(0, ("kerberos_derive_salting_principal: krb5_cc_resolve for %s failed: %s\n", + LIBADS_CCACHE_NAME, error_message(ret))); + return; + } + free_ccache = True; + } + /* Try for each enctype separately, because the rules are * different for different enctypes. */ for (i = 0; enctypes[i] != 0; i++) { @@ -629,6 +640,10 @@ enctypes[i], enctypes); } + + if (free_ccache && ccache) { + krb5_cc_close(context, ccache); + } } /************************************************************************ Modified: trunk/source/libads/kerberos_keytab.c =================================================================== --- trunk/source/libads/kerberos_keytab.c 2004-10-29 22:38:10 UTC (rev 3377) +++ trunk/source/libads/kerberos_keytab.c 2004-10-30 00:34:50 UTC (rev 3378) @@ -102,8 +102,35 @@ /* Construct our principal */ name_to_fqdn(my_fqdn, global_myname()); strlower_m(my_fqdn); - asprintf(&princ_s, "%s/[EMAIL PROTECTED]", srvPrinc, my_fqdn, lp_realm()); + if (strchr_m(srvPrinc, '@')) { + /* It's a fully-named principal. */ + asprintf(&princ_s, "%s", srvPrinc); + } else if (srvPrinc[strlen(srvPrinc)-1] == '$') { + /* It's the machine account, as used by smbclient clients. */ + asprintf(&princ_s, "[EMAIL PROTECTED]", srvPrinc, lp_realm()); + } else { + /* It's a normal service principal. Add the SPN now so that we + * can obtain credentials for it and double-check the salt value + * used to generate the service's keys. */ + asprintf(&princ_s, "%s/[EMAIL PROTECTED]", srvPrinc, my_fqdn, lp_realm()); + /* Update the directory with the SPN */ + DEBUG(3,("ads_keytab_add_entry: Attempting to add/update '%s'\n", princ_s)); + if (!ADS_ERR_OK(ads_add_service_principal_name(ads, global_myname(), srvPrinc))) { + DEBUG(1,("ads_keytab_add_entry: ads_add_service_principal_name failed.\n")); + goto out; + } + } + + ret = get_kerberos_allowed_etypes(context,&enctypes); + if (ret) { + DEBUG(1,("ads_keytab_add_entry: get_kerberos_allowed_etypes failed (%s)\n",error_message(ret))); + goto out; + } + + /* Guess at how the KDC is salting keys for this principal. */ + kerberos_derive_salting_principal(context, NULL, enctypes, princ_s); + ret = krb5_parse_name(context, princ_s, &princ); if (ret) { DEBUG(1,("ads_keytab_add_entry: krb5_parse_name(%s) failed (%s)\n", princ_s, error_message(ret))); @@ -202,12 +229,6 @@ /* If we get here, we have deleted all the old entries with kvno's not equal to the current kvno-1. */ - ret = get_kerberos_allowed_etypes(context,&enctypes); - if (ret) { - DEBUG(1,("ads_keytab_add_entry: get_kerberos_allowed_etypes failed (%s)\n",error_message(ret))); - goto out; - } - /* Now add keytab entries for all encryption types */ for (i = 0; enctypes[i]; i++) { krb5_keyblock *keyp; @@ -242,13 +263,6 @@ krb5_kt_close(context, keytab); keytab = NULL; /* Done with keytab now. No double free. */ - /* Update the LDAP with the SPN */ - DEBUG(3,("ads_keytab_add_entry: Attempting to add/update '%s'\n", princ_s)); - if (!ADS_ERR_OK(ads_add_service_principal_name(ads, global_myname(), srvPrinc))) { - DEBUG(1,("ads_keytab_add_entry: ads_add_service_principcal_name failed.\n")); - goto out; - } - out: SAFE_FREE(principal); @@ -412,8 +426,9 @@ krb5_kt_cursor cursor; krb5_keytab_entry kt_entry; krb5_kvno kvno; + fstring my_fqdn, my_Fqdn, my_name, my_NAME; int i, found = 0; - char **oldEntries = NULL; + char **oldEntries = NULL, *princ_s[18];; ret = ads_keytab_add_entry(ads, "host"); if (ret) { @@ -426,6 +441,45 @@ return ret; } + fstrcpy(my_name, global_myname()); + strlower_m(my_name); + fstrcpy(my_NAME, global_myname()); + strupper_m(my_NAME); + name_to_fqdn(my_Fqdn, global_myname()); + strlower_m(my_Fqdn); + memcpy(my_Fqdn, my_NAME, strlen(my_NAME)); + name_to_fqdn(my_fqdn, global_myname()); + strlower_m(my_fqdn); + + asprintf(&princ_s[0], "[EMAIL PROTECTED]", my_name, lp_realm()); + asprintf(&princ_s[1], "[EMAIL PROTECTED]", my_NAME, lp_realm()); + asprintf(&princ_s[2], "host/[EMAIL PROTECTED]", my_name, lp_realm()); + asprintf(&princ_s[3], "host/[EMAIL PROTECTED]", my_NAME, lp_realm()); + asprintf(&princ_s[4], "host/[EMAIL PROTECTED]", my_fqdn, lp_realm()); + asprintf(&princ_s[5], "host/[EMAIL PROTECTED]", my_Fqdn, lp_realm()); + asprintf(&princ_s[6], "HOST/[EMAIL PROTECTED]", my_name, lp_realm()); + asprintf(&princ_s[7], "HOST/[EMAIL PROTECTED]", my_NAME, lp_realm()); + asprintf(&princ_s[8], "HOST/[EMAIL PROTECTED]", my_fqdn, lp_realm()); + asprintf(&princ_s[9], "HOST/[EMAIL PROTECTED]", my_Fqdn, lp_realm()); + asprintf(&princ_s[10], "cifs/[EMAIL PROTECTED]", my_name, lp_realm()); + asprintf(&princ_s[11], "cifs/[EMAIL PROTECTED]", my_NAME, lp_realm()); + asprintf(&princ_s[12], "cifs/[EMAIL PROTECTED]", my_fqdn, lp_realm()); + asprintf(&princ_s[13], "cifs/[EMAIL PROTECTED]", my_Fqdn, lp_realm()); + asprintf(&princ_s[14], "CIFS/[EMAIL PROTECTED]", my_name, lp_realm()); + asprintf(&princ_s[15], "CIFS/[EMAIL PROTECTED]", my_NAME, lp_realm()); + asprintf(&princ_s[16], "CIFS/[EMAIL PROTECTED]", my_fqdn, lp_realm()); + asprintf(&princ_s[17], "CIFS/[EMAIL PROTECTED]", my_Fqdn, lp_realm()); + + for (i = 0; i < sizeof(princ_s) / sizeof(princ_s[0]); i++) { + if (princ_s[i] != NULL) { + ret = ads_keytab_add_entry(ads, princ_s[i]); + if (ret != 0) { + DEBUG(1,("ads_keytab_create_default: ads_keytab_add_entry failed while adding '%s'.\n", princ_s[i])); + } + SAFE_FREE(princ_s[i]); + } + } + kvno = (krb5_kvno) ads_get_kvno(ads, global_myname()); if (kvno == -1) { DEBUG(1,("ads_keytab_create_default: ads_get_kvno failed to determine the system's kvno.\n")); @@ -497,6 +551,11 @@ * or mb strings into account. Maybe this is because they assume utf8 ? * In this case we may need to convert from utf8 to mb charset here ? JRA. */ + p = strchr_m(ktprinc, '@'); + if (p) { + *p = '\0'; + } + p = strchr_m(ktprinc, '/'); if (p) { *p = '\0'; Modified: trunk/source/libsmb/clikrb5.c =================================================================== --- trunk/source/libsmb/clikrb5.c 2004-10-29 22:38:10 UTC (rev 3377) +++ trunk/source/libsmb/clikrb5.c 2004-10-30 00:34:50 UTC (rev 3378) @@ -77,7 +77,7 @@ pkaddr->contents = (krb5_octet *)&(((struct sockaddr_in *)paddr)->sin_addr); } #else - __ERROR__XX__UNKNOWN_ADDRTYPE +#error UNKNOWN_ADDRTYPE #endif #if defined(HAVE_KRB5_PRINCIPAL2SALT) && defined(HAVE_KRB5_USE_ENCTYPE) && defined(HAVE_KRB5_STRING_TO_KEY) @@ -120,7 +120,7 @@ salt, key); } #else - __ERROR_XX_UNKNOWN_CREATE_KEY_FUNCTIONS +#error UNKNOWN_CREATE_KEY_FUNCTIONS #endif int create_kerberos_key_from_string(krb5_context context,