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

include Mbedtls #48

Merged
merged 5 commits into from
Nov 11, 2018
Merged

Conversation

merlokk
Copy link
Contributor

@merlokk merlokk commented Nov 7, 2018

crypto tests passed
emv works
aes works
sha1, sha256 works

should be ok.

@merlokk
Copy link
Contributor Author

merlokk commented Nov 7, 2018

maybe needs to check cmdflashmem.h
and needs to get rid of polarssl/des.c but in next PR

@merlokk merlokk mentioned this pull request Nov 8, 2018
@merlokk
Copy link
Contributor Author

merlokk commented Nov 8, 2018

as for cmdflashmem.h
it makes a signature and send it to arm. then arm dont write it to flash(
so )
good part: crypto works)
bad part: arm side flash write dont work(

@RfidResearchGroup
Copy link
Collaborator

no, that signature is reserved for RRG, its our detection of a geniue RDV40. So we don't encourage the writing of own signatures. hence not enabled.

Did you pull in the whole crypto-lib? Nice!

@RfidResearchGroup RfidResearchGroup merged commit d2f3a9f into RfidResearchGroup:master Nov 11, 2018
@merlokk merlokk deleted the mbedtls branch November 11, 2018 18:00
@merlokk
Copy link
Contributor Author

merlokk commented Nov 11, 2018

here is mbedtls with some parts is cut. but. they easily can be added with add a files and uncomment lines in config.h
I have added all the parts that are needed for our area (as i see).
i tried to add less code, but here is cmac and error modules that includes everything)
p.s. ecdsa and asn.1 certificates added

iceman1001 pushed a commit that referenced this pull request Sep 28, 2020
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.

None yet

2 participants