s4:heimdal: import lorikeet-heimdal-200906080040 (commit 904d0124b46eed7a8ad6e5b73e89...
[amitay/samba.git] / source4 / heimdal / lib / krb5 / rd_cred.c
index 9129eceeff45de0216c8c22ba70b1a048e1fd1a6..f41edfa2b51dad58f60128878540443291a84807 100644 (file)
@@ -1,40 +1,38 @@
 /*
- * Copyright (c) 1997 - 2005 Kungliga Tekniska Högskolan
- * (Royal Institute of Technology, Stockholm, Sweden). 
- * All rights reserved. 
+ * Copyright (c) 1997 - 2007 Kungliga Tekniska Högskolan
+ * (Royal Institute of Technology, Stockholm, Sweden).
+ * All rights reserved.
  *
- * Redistribution and use in source and binary forms, with or without 
- * modification, are permitted provided that the following conditions 
- * are met: 
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
  *
- * 1. Redistributions of source code must retain the above copyright 
- *    notice, this list of conditions and the following disclaimer. 
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
  *
- * 2. Redistributions in binary form must reproduce the above copyright 
- *    notice, this list of conditions and the following disclaimer in the 
- *    documentation and/or other materials provided with the distribution. 
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
  *
- * 3. Neither the name of the Institute nor the names of its contributors 
- *    may be used to endorse or promote products derived from this software 
- *    without specific prior written permission. 
+ * 3. Neither the name of the Institute nor the names of its contributors
+ *    may be used to endorse or promote products derived from this software
+ *    without specific prior written permission.
  *
- * THIS SOFTWARE IS PROVIDED BY THE INSTITUTE AND CONTRIBUTORS ``AS IS'' AND 
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE 
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE 
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE INSTITUTE OR CONTRIBUTORS BE LIABLE 
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL 
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS 
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) 
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT 
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY 
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF 
- * SUCH DAMAGE. 
+ * THIS SOFTWARE IS PROVIDED BY THE INSTITUTE AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED.  IN NO EVENT SHALL THE INSTITUTE OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
  */
 
 #include <krb5_locl.h>
 
-RCSID("$Id: rd_cred.c,v 1.23 2005/06/17 04:31:48 lha Exp $");
-
 static krb5_error_code
 compare_addrs(krb5_context context,
              krb5_address *a,
@@ -49,7 +47,8 @@ compare_addrs(krb5_context context,
 
     krb5_print_address (a, a_str, sizeof(a_str), &len);
     krb5_print_address (b, b_str, sizeof(b_str), &len);
-    krb5_set_error_string(context, "%s: %s != %s", message, b_str, a_str);
+    krb5_set_error_message(context, KRB5KRB_AP_ERR_BADADDR,
+                          "%s: %s != %s", message, b_str, a_str);
     return KRB5KRB_AP_ERR_BADADDR;
 }
 
@@ -68,64 +67,98 @@ krb5_rd_cred(krb5_context context,
     krb5_crypto crypto;
     int i;
 
-    if ((auth_context->flags & 
+    memset(&enc_krb_cred_part, 0, sizeof(enc_krb_cred_part));
+
+    if ((auth_context->flags &
         (KRB5_AUTH_CONTEXT_RET_TIME | KRB5_AUTH_CONTEXT_RET_SEQUENCE)) &&
        outdata == NULL)
        return KRB5_RC_REQUIRED; /* XXX better error, MIT returns this */
 
     *ret_creds = NULL;
 
-    ret = decode_KRB_CRED(in_data->data, in_data->length, 
+    ret = decode_KRB_CRED(in_data->data, in_data->length,
                          &cred, &len);
-    if(ret)
+    if(ret) {
+       krb5_clear_error_message(context);
        return ret;
+    }
 
     if (cred.pvno != 5) {
        ret = KRB5KRB_AP_ERR_BADVERSION;
-       krb5_clear_error_string (context);
+       krb5_clear_error_message (context);
        goto out;
     }
 
     if (cred.msg_type != krb_cred) {
        ret = KRB5KRB_AP_ERR_MSG_TYPE;
-       krb5_clear_error_string (context);
+       krb5_clear_error_message (context);
        goto out;
     }
 
-    if (cred.enc_part.etype == ETYPE_NULL) {  
+    if (cred.enc_part.etype == ETYPE_NULL) {
        /* DK: MIT GSS-API Compatibility */
        enc_krb_cred_part_data.length = cred.enc_part.cipher.length;
        enc_krb_cred_part_data.data   = cred.enc_part.cipher.data;
     } else {
-       if (auth_context->remote_subkey)
+       /* Try both subkey and session key.
+        *
+        * RFC4120 claims we should use the session key, but Heimdal
+        * before 0.8 used the remote subkey if it was send in the
+        * auth_context.
+        */
+
+       if (auth_context->remote_subkey) {
            ret = krb5_crypto_init(context, auth_context->remote_subkey,
                                   0, &crypto);
-       else
+           if (ret)
+               goto out;
+
+           ret = krb5_decrypt_EncryptedData(context,
+                                            crypto,
+                                            KRB5_KU_KRB_CRED,
+                                            &cred.enc_part,
+                                            &enc_krb_cred_part_data);
+       
+           krb5_crypto_destroy(context, crypto);
+       }
+
+       /*
+        * If there was not subkey, or we failed using subkey,
+        * retry using the session key
+        */
+       if (auth_context->remote_subkey == NULL || ret == KRB5KRB_AP_ERR_BAD_INTEGRITY)
+       {
+
            ret = krb5_crypto_init(context, auth_context->keyblock,
                                   0, &crypto);
-       /* DK: MIT rsh */
 
-       if (ret)
-           goto out;
-       
-       ret = krb5_decrypt_EncryptedData(context,
-                                        crypto,
-                                        KRB5_KU_KRB_CRED,
-                                        &cred.enc_part,
-                                        &enc_krb_cred_part_data);
-       
-       krb5_crypto_destroy(context, crypto);
+           if (ret)
+               goto out;
+       
+           ret = krb5_decrypt_EncryptedData(context,
+                                            crypto,
+                                            KRB5_KU_KRB_CRED,
+                                            &cred.enc_part,
+                                            &enc_krb_cred_part_data);
+       
+           krb5_crypto_destroy(context, crypto);
+       }
        if (ret)
            goto out;
     }
 
-    ret = krb5_decode_EncKrbCredPart (context,
-                                     enc_krb_cred_part_data.data,
-                                     enc_krb_cred_part_data.length,
-                                     &enc_krb_cred_part,
-                                     &len);
-    if (ret)
+    ret = decode_EncKrbCredPart(enc_krb_cred_part_data.data,
+                               enc_krb_cred_part_data.length,
+                               &enc_krb_cred_part,
+                               &len);
+    if (enc_krb_cred_part_data.data != cred.enc_part.cipher.data)
+       krb5_data_free(&enc_krb_cred_part_data);
+    if (ret) {
+       krb5_set_error_message(context, ret,
+                              N_("Failed to decode "
+                                 "encrypte credential part", ""));
        goto out;
+    }
 
     /* check sender address */
 
@@ -141,8 +174,9 @@ krb5_rd_cred(krb5_context context,
            goto out;
 
 
-       ret = compare_addrs(context, a, enc_krb_cred_part.s_address, 
-                           "sender address is wrong in received creds");
+       ret = compare_addrs(context, a, enc_krb_cred_part.s_address,
+                           N_("sender address is wrong "
+                              "in received creds", ""));
        krb5_free_address(context, a);
        free(a);
        if(ret)
@@ -161,9 +195,10 @@ krb5_rd_cred(krb5_context context,
                                      auth_context->local_port);
            if (ret)
                goto out;
-           
-           ret = compare_addrs(context, a, enc_krb_cred_part.r_address, 
-                               "receiver address is wrong in received creds");
+       
+           ret = compare_addrs(context, a, enc_krb_cred_part.r_address,
+                               N_("receiver address is wrong "
+                                  "in received creds", ""));
            krb5_free_address(context, a);
            free(a);
            if(ret)
@@ -171,7 +206,8 @@ krb5_rd_cred(krb5_context context,
        } else {
            ret = compare_addrs(context, auth_context->local_address,
                                enc_krb_cred_part.r_address,
-                               "receiver address is wrong in received creds");
+                               N_("receiver address is wrong "
+                                  "in received creds", ""));
            if(ret)
                goto out;
        }
@@ -187,13 +223,13 @@ krb5_rd_cred(krb5_context context,
            enc_krb_cred_part.usec      == NULL ||
            abs(*enc_krb_cred_part.timestamp - sec)
            > context->max_skew) {
-           krb5_clear_error_string (context);
+           krb5_clear_error_message (context);
            ret = KRB5KRB_AP_ERR_SKEW;
            goto out;
        }
     }
 
-    if ((auth_context->flags & 
+    if ((auth_context->flags &
         (KRB5_AUTH_CONTEXT_RET_TIME | KRB5_AUTH_CONTEXT_RET_SEQUENCE))) {
        /* if these fields are not present in the cred-part, silently
            return zero */
@@ -205,15 +241,16 @@ krb5_rd_cred(krb5_context context,
        if(enc_krb_cred_part.nonce)
            outdata->seq = *enc_krb_cred_part.nonce;
     }
-    
+
     /* Convert to NULL terminated list of creds */
 
-    *ret_creds = calloc(enc_krb_cred_part.ticket_info.len + 1, 
+    *ret_creds = calloc(enc_krb_cred_part.ticket_info.len + 1,
                        sizeof(**ret_creds));
 
     if (*ret_creds == NULL) {
        ret = ENOMEM;
-       krb5_set_error_string (context, "malloc: out of memory");
+       krb5_set_error_message(context, ret,
+                              N_("malloc: out of memory", ""));
        goto out;
     }
 
@@ -224,19 +261,23 @@ krb5_rd_cred(krb5_context context,
        creds = calloc(1, sizeof(*creds));
        if(creds == NULL) {
            ret = ENOMEM;
-           krb5_set_error_string (context, "malloc: out of memory");
+           krb5_set_error_message(context, ret,
+                                  N_("malloc: out of memory", ""));
            goto out;
        }
 
-       ASN1_MALLOC_ENCODE(Ticket, creds->ticket.data, creds->ticket.length, 
+       ASN1_MALLOC_ENCODE(Ticket, creds->ticket.data, creds->ticket.length,
                           &cred.tickets.val[i], &len, ret);
-       if (ret)
+       if (ret) {
+           free(creds);
            goto out;
+       }
        if(creds->ticket.length != len)
            krb5_abortx(context, "internal error in ASN.1 encoder");
        copy_EncryptionKey (&kci->key, &creds->session);
        if (kci->prealm && kci->pname)
-           _krb5_principalname2krb5_principal (&creds->client,
+           _krb5_principalname2krb5_principal (context,
+                                               &creds->client,
                                                *kci->pname,
                                                *kci->prealm);
        if (kci->flags)
@@ -250,7 +291,8 @@ krb5_rd_cred(krb5_context context,
        if (kci->renew_till)
            creds->times.renew_till = *kci->renew_till;
        if (kci->srealm && kci->sname)
-           _krb5_principalname2krb5_principal (&creds->server,
+           _krb5_principalname2krb5_principal (context,
+                                               &creds->server,
                                                *kci->sname,
                                                *kci->srealm);
        if (kci->caddr)
@@ -262,14 +304,20 @@ krb5_rd_cred(krb5_context context,
        
     }
     (*ret_creds)[i] = NULL;
+
+    free_KRB_CRED (&cred);
+    free_EncKrbCredPart(&enc_krb_cred_part);
+
     return 0;
 
   out:
+    free_EncKrbCredPart(&enc_krb_cred_part);
     free_KRB_CRED (&cred);
     if(*ret_creds) {
        for(i = 0; (*ret_creds)[i]; i++)
            krb5_free_creds(context, (*ret_creds)[i]);
        free(*ret_creds);
+       *ret_creds = NULL;
     }
     return ret;
 }