Changeset 1265 for pjproject/trunk/pjlib-util/src/pjlib-util/hmac_md5.c

Timestamp:

May 11, 2007 10:37:14 AM (17 years ago)

Author:

bennylp

Message:

Fixed missing padding when calculating MESSAGE-INTEGRITY in STUN

File:

• pjproject/trunk/pjlib-util/src/pjlib-util/hmac_md5.c (modified) (2 diffs)

pjproject/trunk/pjlib-util/src/pjlib-util/hmac_md5.c

@@ -17 +17 @@
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA 
  */
-#include <pjlib-util/md5.h>
+#include <pjlib-util/hmac_md5.h>
 #include <pj/string.h>
 
 
-/* This code is taken from RFC 2104 */
-
-
-PJ_DEF(void) pj_hmac_md5( const pj_uint8_t *input, unsigned input_len, 
-                          const pj_uint8_t *key, unsigned key_len, 
-                          pj_uint8_t digest[16] )
+PJ_DEF(void) pj_hmac_md5_init(pj_hmac_md5_context *hctx, 
+                              const pj_uint8_t *key, unsigned key_len)
 {
-    pj_md5_context context;
-    pj_uint8_t k_ipad[65];
-    pj_uint8_t k_opad[65];
+    pj_uint8_t k_ipad[64];
     pj_uint8_t tk[16];
     int i;
@@ -46 +40 @@
     }
 
+    /*
+     * HMAC = H(K XOR opad, H(K XOR ipad, text))
+     */
+
     /* start out by storing key in pads */
     pj_bzero( k_ipad, sizeof(k_ipad));
-    pj_bzero( k_opad, sizeof(k_opad));
+    pj_bzero( hctx->k_opad, sizeof(hctx->k_opad));
     pj_memcpy( k_ipad, key, key_len);
-    pj_memcpy( k_opad, key, key_len);
+    pj_memcpy( hctx->k_opad, key, key_len);
 
     /* XOR key with ipad and opad values */
     for (i=0; i<64; i++) {
         k_ipad[i] ^= 0x36;
-        k_opad[i] ^= 0x5c;
+        hctx->k_opad[i] ^= 0x5c;
     }
     /*
      * perform inner MD5
      */
-    pj_md5_init(&context);
-    pj_md5_update(&context, k_ipad, 64);
-    pj_md5_update(&context, input, input_len);
-    pj_md5_final(&context, digest);
+    pj_md5_init(&hctx->context);
+    pj_md5_update(&hctx->context, k_ipad, 64);
+
+}
+
+PJ_DEF(void) pj_hmac_md5_update(pj_hmac_md5_context *hctx,
+                                 const pj_uint8_t *input, 
+                                 unsigned input_len)
+{
+    pj_md5_update(&hctx->context, input, input_len);
+}
+
+PJ_DEF(void) pj_hmac_md5_final(pj_hmac_md5_context *hctx,
+                                pj_uint8_t digest[16])
+{
+    pj_md5_final(&hctx->context, digest);
 
     /*
      * perform outer MD5
      */
-    pj_md5_init(&context);
-    pj_md5_update(&context, k_opad, 64);
-    pj_md5_update(&context, digest, 16);
-    pj_md5_final(&context, digest);
+    pj_md5_init(&hctx->context);
+    pj_md5_update(&hctx->context, hctx->k_opad, 64);
+    pj_md5_update(&hctx->context, digest, 16);
+    pj_md5_final(&hctx->context, digest);
 }
 
+PJ_DEF(void) pj_hmac_md5( const pj_uint8_t *input, unsigned input_len, 
+                          const pj_uint8_t *key, unsigned key_len, 
+                          pj_uint8_t digest[16] )
+{
+    pj_hmac_md5_context ctx;
+
+    pj_hmac_md5_init(&ctx, key, key_len);
+    pj_hmac_md5_update(&ctx, input, input_len);
+    pj_hmac_md5_final(&ctx, digest);
+}
+