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

[WIP]fix compile error for re-introducing exists options. #293

Draft
wants to merge 10 commits into
base: tls13-prototype
Choose a base branch
from

Commits on Jul 23, 2021

  1. Add different options test script

    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 <[email protected]>
    yuhaoth committed Jul 23, 2021
    Configuration menu
    Copy the full SHA
    d855fa9 View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2021

  1. fix unused-function warning

    `write_version` and `read_version` is used under mps
    case. If mps is disabled, the functions should be disabled
    also.
    
    Change-Id: Ib55054eb1e6a0cd0a337ae64105817f025717150
    Signed-off-by: Jerry Yu <[email protected]>
    yuhaoth committed Jul 26, 2021
    Configuration menu
    Copy the full SHA
    a11a793 View commit details
    Browse the repository at this point in the history
  2. fix local variables undefined error

    Remove MBED_SSL_SESSION_TICKETS condition to fix `start` not defined.
    And add TLS1_2_OR_EARLIER to avoid unused variable error
    
    Add MBEDTLS_SSL_RENEGOTIATION condition to fix `ret` not defined error.
    
    Change-Id: I76608d16c61a5135d78d2ad878b09bd6ac5a4899
    Signed-off-by: Jerry Yu <[email protected]>
    yuhaoth committed Jul 26, 2021
    Configuration menu
    Copy the full SHA
    df0e31f View commit details
    Browse the repository at this point in the history

Commits on Jul 27, 2021

  1. fix mbedtls_ssl_check_cert_usage compile fail

    `mbedtls_ssl_check_cert_usage` is redefined in TLS1.3.
    That's due to different of `key_exchange` field. The function
    only use `key_exchange` field of `ciphersuite_info`.
    To keep consistency, we change the prototype of it.
    
    Change-Id: I1905866e3e5dbfbdbff760896fce8b8eb40502c4
    Signed-off-by: Jerry Yu <[email protected]>
    yuhaoth committed Jul 27, 2021
    Configuration menu
    Copy the full SHA
    34d751d View commit details
    Browse the repository at this point in the history
  2. fix functions undeclared warnings

    With TLS1.2 enabled, those functions report
    undeclared warning
    
    Change-Id: Ie20e9e9e9cee3fe8561c368c24042096b0b36320
    Signed-off-by: Jerry Yu <[email protected]>
    yuhaoth committed Jul 27, 2021
    Configuration menu
    Copy the full SHA
    81c88ff View commit details
    Browse the repository at this point in the history
  3. fix sig_hashes config error

    In config default function, sig_hashes is configed to diferent
    value according to TLS vesrion.
    
    To support TLS1.2 and 1.3, set different value according to version
    config.
    
    Change-Id: Ibef8e05b1453d2f0fc522eaf5b9ec370a84829e2
    Signed-off-by: Jerry Yu <[email protected]>
    yuhaoth committed Jul 27, 2021
    Configuration menu
    Copy the full SHA
    bd8e483 View commit details
    Browse the repository at this point in the history

Commits on Jul 28, 2021

  1. Add conditional compilation for mbedtls_ssl_conf_dtls_cookies

    mbedtls_ssl_conf_dtls_cookies depends on MBEDTLS_SSL_DTLS_HELLO_VERIFY.
    Not all place is wrapped with it.
    
    CustomizedGitHooks: yes
    Change-Id: I603cbaeabccf969c2785198409c0d59f3afa889f
    Signed-off-by: Jerry Yu <[email protected]>
    yuhaoth committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    17e162b View commit details
    Browse the repository at this point in the history
  2. Merge remote-tracking branches 'origin/pr/add-options-test-script', '…

    …origin/pr/fix-warnings', 'origin/pr/fix-local-variable-undefine-error', 'origin/pr/fix-mbedtls_ssl_check_cert_usage-error', 'origin/pr/fix-functions-undeclared-warnings' and 'origin/pr/fix-sig_hashes-errors' into tls13-prototype
    
    Change-Id: Ic6ee391e3df9133c8c35ad3554b9571d558c1048
    yuhaoth committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    c494232 View commit details
    Browse the repository at this point in the history
  3. disable EXPORT_KEYS

    Change-Id: I63e218070e96637a15242fec3a66b5e448986287
    CustomizedGitHooks: yes
    yuhaoth committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    8917997 View commit details
    Browse the repository at this point in the history
  4. Enable MBEDTLS_SSL_PROTO_*

    First step to enable other protocol.
    Compile status: Fail
    
    Change-Id: I9fbb2e7994abe556db3954e69807f45b0f99d63e
    CustomizedGitHooks: yes
    Signed-off-by: Jerry Yu <[email protected]>
    yuhaoth committed Jul 28, 2021
    Configuration menu
    Copy the full SHA
    2306e50 View commit details
    Browse the repository at this point in the history