Skip to content

Conversation

@xaqfan
Copy link
Contributor

@xaqfan xaqfan commented Apr 25, 2022

To support feitian epass2003 keys with cos of fips.

Copy link
Member

@Jakuje Jakuje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there are also many formatting problems. Please, review the surrounding code for reference how the code should be formatted, mostly around braces. Remove any space indentation. I mentioned couple of comments inline.

To provide updated changes, you do not have to create a new PR, just push into the current branch. The PR will get updated.

@Jakuje
Copy link
Member

Jakuje commented May 4, 2022

please, rebase on current master to get working CI results also for openssl 3.0 and libressl.

xaqfan added 2 commits May 5, 2022 13:22
indent and replace memset  in  some where
Indent and CMAC interfaces of different OpenSSL versions
Copy link
Contributor Author

@xaqfan xaqfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indent and cmac interfaces of different OpenSSL versions

xaqfan added 2 commits May 6, 2022 09:55
indent and no memset to set one byte.
Copy link
Contributor Author

@xaqfan xaqfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

repair indent of lines

repair if  indent  in  aes128_encrypt_cmac  function
Copy link
Contributor Author

@xaqfan xaqfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

repair if indent in aes128_encrypt_cmac function

Merge if statement blocks
Copy link
Contributor Author

@xaqfan xaqfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge if statement blocks

@xaqfan
Copy link
Contributor Author

xaqfan commented May 9, 2022

@frankmorgner
is there anything need change?

Thanks @Jakuje @frankmorgner

@xaqfan xaqfan requested a review from frankmorgner May 20, 2022 08:01
Copy link
Contributor Author

@xaqfan xaqfan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No reply for a long time.
Submit it again.

@Jakuje Jakuje merged commit 5d6f7cb into OpenSC:master May 22, 2022
@Jakuje
Copy link
Member

Jakuje commented May 22, 2022

Thank you. I think we are good.

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.

4 participants