fix computeMac

This commit is contained in:
Kyle Spearrin 2017-01-13 00:22:42 -05:00
parent 3728cd8e1d
commit 65d822716e
1 changed files with 7 additions and 8 deletions

View File

@ -292,21 +292,21 @@ function initCryptoService() {
throw 'MAC key unavailable.'; throw 'MAC key unavailable.';
} }
var ivBits = forge.util.decode64(cipherString.initializationVector); var ivBytes = forge.util.decode64(cipherString.initializationVector);
var ctBits = forge.util.decode64(cipherString.cipherText); var ctBytes = forge.util.decode64(cipherString.cipherText);
var computedMac = null; var computedMac = null;
if (cipherString.mac) { if (cipherString.mac) {
computedMac = computeMac(ctBits, ivBits, macKey); computedMac = computeMac(ctBytes, ivBytes, macKey);
if (computedMac !== cipherString.mac) { if (computedMac !== cipherString.mac) {
console.error('MAC failed.'); console.error('MAC failed.');
deferred.reject('MAC failed.'); deferred.reject('MAC failed.');
} }
} }
var ctBuffer = forge.util.createBuffer(ctBits); var ctBuffer = forge.util.createBuffer(ctBytes);
var decipher = forge.cipher.createDecipher('AES-CBC', computedMac ? theEncKey : key); var decipher = forge.cipher.createDecipher('AES-CBC', computedMac ? theEncKey : key);
decipher.start({ iv: ivBits }); decipher.start({ iv: ivBytes });
decipher.update(ctBuffer); decipher.update(ctBuffer);
decipher.finish(); decipher.finish();
@ -320,11 +320,10 @@ function initCryptoService() {
}; };
function computeMac(ct, iv, macKey) { function computeMac(ct, iv, macKey) {
var bits = iv.concat(ct);
var hmac = forge.hmac.create(); var hmac = forge.hmac.create();
hmac.start('sha256', macKey); hmac.start('sha256', macKey);
hmac.update(bits); hmac.update(iv + ct);
var mac = hmac.digest(); var mac = hmac.digest();
return forge.util.encode64(mac); return forge.util.encode64(mac.getBytes());
} }
}; };