Add AES encrypt/decrypt support for SE05X on Arduino with two example sketches

[jowin202]

Hello,

I used the se05x_APDU_apis.h function for block cipher to implement a demonstration example sketch for AES encryption and decryption. I found AES encryption with secure element nowhere (looked for days/hours) and decieded to do it by myself.

Please add this

kind regards

[CLAassistant]

All committers have signed the CLA.

[jowin202]

OK, the building process failed with

'class SE05XClass' has no member named 'AES_ECB_encrypt'
'class SE05XClass' has no member named 'AES_ECB_decrypt'
'class SE05XClass' has no member named 'writeAESKey'; did you mean 'writeSlot'?

but this are exactly the 3 functions I added to SE05X.c / SE05X.h library.

[facchinm]

@jowin202 test failures on the stable version arduino:renesas_portenta:portenta_c33 are expected (since the APIs are not there yet 😉 ). The test that shouldn't fail (and passes indeed) is on arduino-git:renesas:portenta_c33
Waiting for @pennam review but it looks good to me 👍

[pennam]

Tested and works as expected. Thanks @jowin202

[jowin202]

Hello,
Just a question: Will this pull request be merged into the original repository (and when)? I would like to do another pull request for HMAC support, but I'm not familiar with github and the branch-things. So I would like to clear my fork and make a completely new pull request.
Kind regards

[pennam]

Hi @jowin202
I see you have added a commit here. Is this the HMAC support you are referring to ? Or do you want to add more changes ?

See commit 336826e

[jowin202]

Oh I see the changes are already here now. Yes, I added HMAC to my personal fork, but I don't know if this is working and ready for being included into the main branch.

[pennam]

Ok, so i will take your first commit open a new pr and merge it into main. After that when code is ready you can open a new PR for HMAC support.

[jowin202]

Ahh ok. I have 2 commits in this pull request

add AES: 2f96751
add HMAC: 336826e

Hopefully they can be included into the arduino lib

[pennam]

is also 336826e ready? It was not included in the original PR and i've understood you are not sure it is working.

Yes, I added HMAC to my personal fork, but I don't know if this is working and ready for being included into the main branch.

[jowin202]

is also 336826e ready? It was not included in the original PR and i've understood you are not sure it is working.

Yes, I added HMAC to my personal fork, but I don't know if this is working and ready for being included into the main branch.

It’s also ready from my side. I haven’t figured out how to make a new pull request.

[pennam]

superseded by #242