commit ce64d9f50f6b1358f6c672ffcff85bce126042a1
author Chia-chi Yeh <chiachi@android.com> Tue Dec 29 21:36:43 2009 -0800
committer Android (Google) Code Review <android-gerrit@google.com> Tue Dec 29 21:36:43 2009 -0800
tree e425040788da8fdb4f86916a07f999534a4938a0
parent f028eaa35be37c9f974668c2da9aa3a6a3d634f4
parent b78679e18db941fa0c104d728db128cf9ee21dc7

Merge "keystore: compute the padding in an intuitive way."

cmds/keystore/keystore.c

diff --git a/cmds/keystore/keystore.c b/cmds/keystore/keystore.c
index 37155e4..4426874 100644
--- a/cmds/keystore/keystore.c
+++ b/cmds/keystore/keystore.c
@@ -170,16 +170,12 @@
         return SYSTEM_ERROR;
     }
 
-    length = (blob.length + blob.value - blob.encrypted) % AES_BLOCK_SIZE;
-    if (length) {
-        length = AES_BLOCK_SIZE - length;
-    }
+    length = blob.length + blob.value - blob.encrypted;
+    length = (length + AES_BLOCK_SIZE - 1) / AES_BLOCK_SIZE * AES_BLOCK_SIZE;
 
-    length += blob.length + blob.value - blob.digested;
     blob.length = htonl(blob.length);
-    MD5(blob.digested, length, blob.digest);
+    MD5(blob.digested, length - (blob.digested - blob.encrypted), blob.digest);
 
-    length += blob.digested - blob.encrypted;
     memcpy(vector, blob.vector, AES_BLOCK_SIZE);
     AES_cbc_encrypt(blob.encrypted, blob.encrypted, length, aes_key, vector,
                     AES_ENCRYPT);