Search code examples
pythoncryptographyhmacsha

problem implementing HMAC-SHA1 giving wrong Hash?


I'm trying to implement my own HMAC-SHA1 function in python but It seems to always give me the wrong checksum at the end which I Can't see why

""" An Hmac Implementation of SHA-1 """
from Crypto.Hash import SHA1

hasher = SHA1.new()

password = b"test"
message = b"NAME"

pad_length = 64 - len(password)
key = bytes(pad_length) + password

ipad_num = "01011100" * 64
opad_num = "00110110" * 64

ipad = int.from_bytes(key, byteorder="big") ^ int(ipad_num, base=2)
opad = int.from_bytes(key, byteorder="big") ^ int(opad_num, base=2)

ipad = int.to_bytes(ipad, length=64, byteorder="big")
opad = int.to_bytes(opad, length=64, byteorder="big")

hasher.update(ipad + message)
inner_hash = hasher.digest()
print("inner hash {}".format(inner_hash.hex()))

hasher = SHA1.new()
hasher.update(opad + inner_hash)
print("final hash {}".format(hasher.hexdigest()))

which is supposed to give me this checksum: for message = NAME password = test

3e0f1cc6c2d787afe49345986212f60d3d4d300d

but instead it give me this checksum

7d6b1ba137a44ee9e083d8e3ba5a84fd739751f4

Solution

  • I would recommend using hmac library python provides unless you are learning cryptography.

    Here's the fixed code:

    """ An Hmac Implementation of SHA-1 """
    from Crypto.Hash import SHA1
    
    hasher = SHA1.new()
    
    password = b"test"
    message = b"NAME"
    
    pad_length = 64 - len(password) # TODO: support longer key
    key = password + bytes(pad_length) # padding should be on the right
    
    ipad_num = "00110110" * 64 # ipad should be 0x36
    opad_num = "01011100" * 64 # opad should be 0x5c
    
    ipad = int.from_bytes(key, byteorder="big") ^ int(ipad_num, base=2)
    opad = int.from_bytes(key, byteorder="big") ^ int(opad_num, base=2)
    
    ipad = int.to_bytes(ipad, length=64, byteorder="big")
    opad = int.to_bytes(opad, length=64, byteorder="big")
    
    hasher.update(ipad + message)
    inner_hash = hasher.digest()
    print("inner hash {}".format(inner_hash.hex()))
    
    hasher = SHA1.new()
    hasher.update(opad + inner_hash)
    print("final hash {}".format(hasher.hexdigest()))