[src/netbsd-1-5]: src Pull up revision 1.3 (requested by assar):

[he <he%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/8eaf80db120a
branches:  netbsd-1-5
changeset: 491146:8eaf80db120a
user:      he <he%NetBSD.org@localhost>
date:      Thu Apr 05 23:24:53 2001 +0000

description:
Pull up revision 1.3 (requested by assar):
  Upgrade Heimdal to version 0.3e.

diffstat:

 crypto/dist/heimdal/kdc/524.c                      |  245 +++++++++++++++-----
 crypto/dist/heimdal/lib/roken/make-print-version.c |    4 +-
 lib/libcom_err/Makefile                            |    4 +-
 lib/libcom_err/shlib_version                       |    2 +-
 lib/libkafs/shlib_version                          |    2 +-
 lib/libss/shlib_version                            |    2 +-
 6 files changed, 186 insertions(+), 73 deletions(-)

diffs (truncated from 365 to 300 lines):

diff -r 6a69c2a6fe72 -r 8eaf80db120a crypto/dist/heimdal/kdc/524.c
--- a/crypto/dist/heimdal/kdc/524.c     Thu Apr 05 23:24:49 2001 +0000
+++ b/crypto/dist/heimdal/kdc/524.c     Thu Apr 05 23:24:53 2001 +0000
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1997-1999 Kungliga Tekniska Högskolan
+ * Copyright (c) 1997-2001 Kungliga Tekniska Högskolan
  * (Royal Institute of Technology, Stockholm, Sweden). 
  * All rights reserved. 
  *
@@ -33,17 +33,158 @@
 
 #include "kdc_locl.h"
 
-RCSID("$Id: 524.c,v 1.1.1.1.2.1 2000/10/17 23:11:27 tv Exp $");
+RCSID("$Id: 524.c,v 1.1.1.1.2.2 2001/04/05 23:24:53 he Exp $");
 
 #ifdef KRB4
 
+/*
+ * fetch the server from `t', returning the name in malloced memory in
+ * `spn' and the entry itself in `server'
+ */
+
+static krb5_error_code
+fetch_server (const Ticket *t,
+             char **spn,
+             hdb_entry **server,
+             const char *from)
+{
+    krb5_error_code ret;
+    krb5_principal sprinc;
+
+    ret = principalname2krb5_principal(&sprinc, t->sname, t->realm);
+    if (ret) {
+       kdc_log(0, "principalname2krb5_principal: %s",
+               krb5_get_err_text(context, ret));
+       return ret;
+    }
+    ret = krb5_unparse_name(context, sprinc, spn);
+    if (ret) {
+       krb5_free_principal(context, sprinc);
+       kdc_log(0, "krb5_unparse_name: %s", krb5_get_err_text(context, ret));
+       return ret;
+    }
+    ret = db_fetch(sprinc, server);
+    krb5_free_principal(context, sprinc);
+    if (ret) {
+       kdc_log(0,
+       "Request to convert ticket from %s for unknown principal %s: %s",
+               from, *spn, krb5_get_err_text(context, ret));
+       if (ret == ENOENT)
+           ret = KRB5KDC_ERR_S_PRINCIPAL_UNKNOWN;
+       return ret;
+    }
+    return 0;
+}
+
+static krb5_error_code
+log_524 (const EncTicketPart *et,
+        const char *from,
+        const char *spn)
+{
+    krb5_principal client;
+    char *cpn;
+    krb5_error_code ret;
+
+    ret = principalname2krb5_principal(&client, et->cname, et->crealm);
+    if (ret) {
+       kdc_log(0, "principalname2krb5_principal: %s",
+               krb5_get_err_text (context, ret));
+       return ret;
+    }
+    ret = krb5_unparse_name(context, client, &cpn);
+    if (ret) {
+       krb5_free_principal(context, client);
+       kdc_log(0, "krb5_unparse_name: %s",
+               krb5_get_err_text (context, ret));
+       return ret;
+    }
+    kdc_log(1, "524-REQ %s from %s for %s", cpn, from, spn);
+    free(cpn);
+    krb5_free_principal(context, client);
+    return 0;
+}
+
+static krb5_error_code
+verify_flags (const EncTicketPart *et,
+             const char *spn)
+{
+    if(et->endtime < kdc_time){
+       kdc_log(0, "Ticket expired (%s)", spn);
+       return KRB5KRB_AP_ERR_TKT_EXPIRED;
+    }
+    if(et->flags.invalid){
+       kdc_log(0, "Ticket not valid (%s)", spn);
+       return KRB5KRB_AP_ERR_TKT_NYV;
+    }
+    return 0;
+}
+
+/*
+ * set the `et->caddr' to the most appropriate address to use, where
+ * `addr' is the address the request was received from.
+ */
+
+static krb5_error_code
+set_address (EncTicketPart *et,
+            struct sockaddr *addr,
+            const char *from)
+{
+    krb5_error_code ret;
+    krb5_address *v4_addr;
+
+    v4_addr = malloc (sizeof(*v4_addr));
+    if (v4_addr == NULL)
+       return ENOMEM;
+
+    ret = krb5_sockaddr2address(addr, v4_addr);
+    if(ret) {
+       free (v4_addr);
+       kdc_log(0, "Failed to convert address (%s)", from);
+       return ret;
+    }
+           
+    if (et->caddr && !krb5_address_search (context, v4_addr, et->caddr)) {
+       kdc_log(0, "Incorrect network address (%s)", from);
+       krb5_free_address(context, v4_addr);
+       free (v4_addr);
+       return KRB5KRB_AP_ERR_BADADDR;
+    }
+    if(v4_addr->addr_type == KRB5_ADDRESS_INET) {
+       /* we need to collapse the addresses in the ticket to a
+          single address; best guess is to use the address the
+          connection came from */
+       
+       if (et->caddr != NULL) {
+           free_HostAddresses(et->caddr);
+       } else {
+           et->caddr = malloc (sizeof (*et->caddr));
+           if (et->caddr == NULL) {
+               krb5_free_address(context, v4_addr);
+               free(v4_addr);
+               return ENOMEM;
+           }
+       }
+       et->caddr->val = v4_addr;
+       et->caddr->len = 1;
+    } else {
+       krb5_free_address(context, v4_addr);
+       free(v4_addr);
+    }
+    return 0;
+}
+
+/*
+ * process a 5->4 request, based on `t', and received `from, addr',
+ * returning the reply in `reply'
+ */
+
 krb5_error_code
-do_524(Ticket *t, krb5_data *reply, const char *from, struct sockaddr *addr)
+do_524(const Ticket *t, krb5_data *reply,
+       const char *from, struct sockaddr *addr)
 {
     krb5_error_code ret = 0;
-    krb5_principal sprinc = NULL;
     krb5_crypto crypto;
-    hdb_entry *server;
+    hdb_entry *server = NULL;
     Key *skey;
     krb5_data et_data;
     EncTicketPart et;
@@ -53,21 +194,29 @@
     unsigned char buf[MAX_KTXT_LEN + 4 * 4];
     size_t len;
     
-    principalname2krb5_principal(&sprinc, t->sname, t->realm);
-    krb5_unparse_name(context, sprinc, &spn);
-    server = db_fetch(sprinc);
-    if(server == NULL){
-       kdc_log(0, "Request to convert ticket from %s for unknown principal %s",
-               from, spn);
+    if(!enable_524) {
+       ret = KRB5KDC_ERR_POLICY;
+       kdc_log(0, "Rejected ticket conversion request from %s", from);
        goto out;
     }
+
+    ret = fetch_server (t, &spn, &server, from);
+    if (ret) {
+       goto out;
+    }
+
     ret = hdb_enctype2key(context, server, t->enc_part.etype, &skey);
     if(ret){
        kdc_log(0, "No suitable key found for server (%s) "
                "when converting ticket from %s", spn, from);
        goto out;
     }
-    krb5_crypto_init(context, &skey->key, 0, &crypto);
+    ret = krb5_crypto_init(context, &skey->key, 0, &crypto);
+    if (ret) {
+       kdc_log(0, "krb5_crypto_init failed: %s",
+               krb5_get_err_text(context, ret));
+       goto out;
+    }
     ret = krb5_decrypt_EncryptedData (context,
                                      crypto,
                                      KRB5_KU_TICKET,
@@ -85,66 +234,32 @@
        kdc_log(0, "Failed to decode ticket from %s for %s", from, spn);
        goto out;
     }
-    {
-       krb5_principal client;
-       char *cpn;
-       principalname2krb5_principal(&client, et.cname, et.crealm);
-       krb5_unparse_name(context, client, &cpn);
-       kdc_log(1, "524-REQ %s from %s for %s", cpn, from, spn);
-       free(cpn);
-       krb5_free_principal(context, client);
-    }
 
-    if(et.endtime < kdc_time){
-       kdc_log(0, "Ticket expired (%s)", spn);
+    ret = log_524 (&et, from, spn);
+    if (ret) {
        free_EncTicketPart(&et);
-       ret = KRB5KRB_AP_ERR_TKT_EXPIRED;
-       goto out;
-    }
-    if(et.flags.invalid){
-       kdc_log(0, "Ticket not valid (%s)", spn);
-       free_EncTicketPart(&et);
-       ret = KRB5KRB_AP_ERR_TKT_NYV;
        goto out;
     }
-    {
-       krb5_addresses *save_caddr, new_addr;
-       krb5_address v4_addr;
+
+    ret = verify_flags (&et, spn);
+    if (ret) {
+       free_EncTicketPart(&et);
+       goto out;
+    }
 
-       ret = krb5_sockaddr2address(addr, &v4_addr);
-       if(ret) {
-           kdc_log(0, "Failed to convert address (%s)", spn);
-           free_EncTicketPart(&et);
-           goto out;
-       }
-           
-       if (et.caddr && !krb5_address_search (context, &v4_addr, et.caddr)) {
-           kdc_log(0, "Incorrect network address (%s)", spn);
-           free_EncTicketPart(&et);
-           krb5_free_address(context, &v4_addr);
-           ret = KRB5KRB_AP_ERR_BADADDR;
-           goto out;
-       }
-       if(v4_addr.addr_type == KRB5_ADDRESS_INET) {
-           /* we need to collapse the addresses in the ticket to a
-              single address; best guess is to use the address the
-              connection came from */
-           save_caddr = et.caddr;
-           new_addr.len = 1;
-           new_addr.val = &v4_addr;
-           et.caddr = &new_addr;
-       }
-       ret  = encode_v4_ticket(buf + sizeof(buf) - 1, sizeof(buf),
-                               &et, &t->sname, &len);
-       if(v4_addr.addr_type == KRB5_ADDRESS_INET)
-           et.caddr = save_caddr;
+    ret = set_address (&et, addr, from);
+    if (ret) {
+       free_EncTicketPart(&et);
+       goto out;
     }
+    ret = encode_v4_ticket(buf + sizeof(buf) - 1, sizeof(buf),
+                          &et, &t->sname, &len);
     free_EncTicketPart(&et);
     if(ret){
        kdc_log(0, "Failed to encode v4 ticket (%s)", spn);
        goto out;
     }
-    ret = get_des_key(server, &skey);
+    ret = get_des_key(server, FALSE, &skey);
     if(ret){
        kdc_log(0, "No DES key for server (%s)", spn);
        goto out;
@@ -173,11 +288,9 @@
     
     if(spn)
        free(spn);
-    if(sprinc)
-       krb5_free_principal(context, sprinc);
-    hdb_free_entry(context, server);
-    free(server);