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.

Sign up for GitHub

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

Jump to bottom

Verify and fix different options: disable MBEDTLS_SSL_USE_MPS, enable MBEDTLS_PROTO_* and nbio=0 #297

Open
yuhaoth opened this issue Jul 20, 2021 · 0 comments
Open

Verify and fix different options: disable MBEDTLS_SSL_USE_MPS, enable MBEDTLS_PROTO_* and nbio=0 #297

yuhaoth opened this issue Jul 20, 2021 · 0 comments
Assignees
@yuhaoth
Labels
bug

Comments

@yuhaoth
Copy link
Collaborator

yuhaoth commented Jul 20, 2021 •
edited
Loading

Description

This is breakdown issue from #15 .

With below configuration , the build fail

#undef MBEDTLS_SSL_USE_MPS
#define MBEDTLS_SSL_PROTO_TLS1_2
@yuhaoth yuhaoth mentioned this issue Jul 21, 2021
Open
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 21, 2021
@yuhaoth
Add different options test script
52ee126 
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <jerry.h.yu@arm.com>
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 21, 2021
@yuhaoth
Add different options test script
d12d784 
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298,hannestschofenig#301

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <jerry.h.yu@arm.com>
@yuhaoth yuhaoth added the bug label Jul 21, 2021
@yuhaoth yuhaoth self-assigned this Jul 21, 2021
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 22, 2021
@yuhaoth
Add different options test script
f51b47c 
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298,hannestschofenig#301

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <jerry.h.yu@arm.com>
yuhaoth added a commit to yuhaoth/mbedtls1.3 that referenced this issue Jul 23, 2021
@yuhaoth
Add different options test script
d855fa9 
According to hannestschofenig#15, different options report
fail. To fix the issue we should not break
passed options. The script is to make sure
current status.

If all relative issues are resolved, This
patch should be removed or re-considered.

issues: hannestschofenig#15, hannestschofenig#297,hannestschofenig#238,hannestschofenig#298,hannestschofenig#301

Change-Id: Iaebbdaa5861802f2a48e6bca238a94672ddfaf70
CustomizedGitHooks: yes
Signed-off-by: Jerry Yu <jerry.h.yu@arm.com>
This was referenced Jul 27, 2021
Merged
Merged
Merged
@yuhaoth yuhaoth mentioned this issue Aug 4, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuhaoth yuhaoth

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yuhaoth