Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement CMAC #377

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Implement CMAC #377

wants to merge 1 commit into from

Conversation

a-dma
Copy link

@a-dma a-dma commented Jul 6, 2016

I was playing around with CMAC and I thought I'd contribute back what code I have.

I'm still learning rust, so the code is by no means perfect. Also, given the lack of documentation there was a bit of guesswork to find things that were already implemented.

A couple of possible (but for me tricky) improvements are:

  • Use a FixedBuffer for storing the MAC result and take advantage of StandardPadding;
  • Try to shoehorn in a CbcEncryptor with an empty IV rather than doing the XOR'ing manually.

Add a generic implementation of CMAC using a BlockEncryptor.
Tested only against AES128, AES192 and AES256.
@flybayer
Copy link

Any chance this will be merged to master soon?

@flybayer
Copy link

@DaGenix is anything besides the failing build preventing this from being merged?

@tvladyslav
Copy link

@flybayer last commit was merged 7 month ago. It smells like a dead repo. I thought about making one small PR, but now I have doubts...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants