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

Tinycrypt SHA256 test blocks at test vector 13 and 14 with microkernel #2637

Closed
zephyrbot opened this issue Oct 27, 2016 · 5 comments
Closed
Labels
area: Security Security bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@zephyrbot
Copy link
Collaborator

zephyrbot commented Oct 27, 2016

Reported by ethan gao:

When test tinycrypt sha256 using test_sha256 underneath tests/crypto of zephyr, the test blocks at test vector 13 and the same to the test vector 14.

[How to reproduce]

  1. checkout zephyr project and cd tests/crypto/test_sha256
  2. run "make pristine && make BOARD=quark_se_c1000_devboard flash"
  3. watch the serial output

[Error log]
tc_start() - Performing SHA256 tests (NIST tests vectors):
SHA256 test #1:

PASS - test_1.
SHA256 test #2:

PASS - test_2.
SHA256 test #3:

PASS - test_3.
SHA256 test #4:

PASS - test_4.
SHA256 test #5:

PASS - test_5.
SHA256 test #6:

PASS - test_6.
SHA256 test #7:

PASS - test_7.
SHA256 test #8:

PASS - test_8.
SHA256 test #9:

PASS - test_9.
SHA256 test #10:

PASS - test_10.
SHA256 test #11:

PASS - test_11.
SHA256 test #12:

PASS - test_12.
SHA256 test #13:

(Imported from Jira ZEP-1142)

@zephyrbot
Copy link
Collaborator Author

by Andrew Boie:

ethan gao this test takes a long time especially the last steps, how long did you let it run?

@zephyrbot
Copy link
Collaborator Author

by Flavio Santes:

Thank you Andrew for your feedback.

ethan gao , make pristine && make && time make qemu gives me the following:

real 3m49.068s
user 3m40.384s
sys 0m1.016s

So far, I don't have access to the quark_se_c1000 dev board, could you re-check?

@zephyrbot
Copy link
Collaborator Author

by ethan gao:

yeah, I had a try keeping it running and back to check the result the second day morning, I saw it passed, As per the Andrew said it takes a long time.

@zephyrbot
Copy link
Collaborator Author

by ethan gao:

because it works finally with a long time required, here to close this ticket.

@zephyrbot
Copy link
Collaborator Author

by Mark Linkmeyer:

Correcting the priority field

@zephyrbot zephyrbot added priority: medium Medium impact/importance bug area: Security Security bug The issue is a bug, or the PR is fixing a bug labels Sep 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Security Security bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

1 participant