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
[mbedtls] update to 2.14.0 #3338
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
556d7d9 Merge pull request openthread#532 from sbutcher-arm/version-2.14.0 c1b9892 Update library version number to 2.14.0 b35e59d Refine the language in the ChangeLog 681edee Fix language and formatting in ChangeLog 06d80cf Fix merge of Changelog 128bit CTR_DRBG entry de13963 Merge remote-tracking branch 'restricted/pr/520' into development-restricted-proposed cdd1a6c Merge remote-tracking branch 'restricted/pr/510' into development-restricted-proposed 2ab14bb Clarified new platform errors in the ChangeLog e51d4b3 Merge remote-tracking branch 'public/pr/2054' into development-proposed 6aa9fb4 Add ChangeLog entry 9924bdc Deprecate hardware acceleration errors bcca58c Add common feature unavailable error d83448b Merge remote-tracking branch 'public/pr/2052' into development-restricted-proposed 53b45ec Merge remote-tracking branch 'public/pr/2079' into development-restricted-proposed bbc31b4 Merge remote-tracking branch 'public/pr/2085' into development-restricted-proposed 93a9b49 Merge remote-tracking branch 'public/pr/2077' into development-restricted-proposed 361ce6c Merge remote-tracking branch 'public/pr/2127' into development-restricted-proposed c818131 Merge remote-tracking branch 'public/pr/2140' into development-restricted-proposed 241823a Merge remote-tracking branch 'public/pr/1641' into development-restricted-proposed 42ab4ae Merge remote-tracking branch 'public/pr/2167' into development-restricted-proposed 51b6abb Merge remote-tracking branch 'public/pr/2165' into development-proposed f143a78 Adapt ChangeLog 7213744 Fix typo in comment 2981d8f Change to positive flow for all cases 93ace01 Revert positive flow check b430d9f Change to positive checks 19779c4 Some style and documentation fixes 5ed8c1e Avoid using restartable and alternative ECP imp. d3445da Adapt ChangeLog b75ffb5 Don't perform binary comparison of CRL issuer and CA subject 1f8527f Move static x509_name_cmp() in library/x509_crt.c a8b13d7 Add tests for relaxed CRL-CA name comparison 06f88e9 Merge remote-tracking branch 'public/pr/2007' into development-proposed 76646a4 Merge remote-tracking branch 'public/pr/1777' into development-proposed 2705bea Merge remote-tracking branch 'public/pr/2095' into development-proposed 710f203 Merge branch 'iotssl-1770' into development_thomas_dee 5517755 Improve wording and formatting of ASN.1 write module documentation cec1c26 Break overly long line in library/x509_create.c ee334a3 Remove Doxygen tags from documentation of private structure beffcd8 Update hardcoded certificates in library/certs.c 6e1adee Regenerate test files 381c77c Change serial in test-ca.crt from 0 to 3 to circumvent ASN.1 bug b12fd31 Adapt ChangeLog cb93813 Don't perform binary comparison of CRL issuer and CA subject 0f6903d Move static x509_name_cmp() in library/x509_crt.c 52acdb5 Add tests for relaxed CRL-CA name comparison b963081 Generate tests/data_files/test-ca_cat[12|21].crt from Makefile 386f99c Generate cli-rsa* CSRs and CRTs through Mbed TLS applications 0dd1139 Generate server5.req.ku.sha1 through Mbed TLS application ebc1f40 Generate server2* CSRs and CRTs through Mbed TLS applications b837775 Generate test-ca* CSRs and CRTs through Mbed TLS applications 50cb93a Generate server1* CRTs and CSRs through Mbed TLS applications 56e8463 Add 'password' cmd line parameter to cert_req example program f745733 Add 'md' cmd line parameter to cert_req example program 2b161c3 Fix compilation issue 936d284 Minor fixes 8493f80 conditionaly compile ECDH and ECDSA alt functions 2b5be1e Fix ChangeLog entry for openthread#2069 0bb204c Adapt ChangeLog 7904f94 Merge remote-tracking branch 'public/pr/1099' into development-proposed 4a865ef Add ChangeLog entry for PR openthread#1618 - ARM DSP instruction support d5bf428 Merge remote-tracking branch 'public/pr/1618' into development-proposed 5706452 Merge remote-tracking branch 'public/pr/2056' into development-proposed b4e5451 Merge remote-tracking branch 'public/pr/2070' into development-proposed 17a0fab Merge remote-tracking branch 'public/pr/2111' into development-proposed 02ef525 Merge remote-tracking branch 'public/pr/2107' into development-proposed a07d86e Merge remote-tracking branch 'public/pr/1902' into development ef263eb Merge remote-tracking branch 'public/pr/1993' into development 3869384 Merge remote-tracking branch 'public/pr/2031' into development da09561 Merge remote-tracking branch 'public/pr/2142' into development ddc9e26 Merge remote-tracking branch 'public/pr/1627' into development 78dd2e5 Merge remote-tracking branch 'public/pr/1806' into development 1daecb4 Merge remote-tracking branch 'public/pr/1281' into development e783f4a Merge remote-tracking branch 'public/pr/1609' into development b10c660 Detect unsigned integer overflow in mbedtls_ecp_check_budget() abdf67e Cast number of operations to `uint` in MBEDTLS_ECP_BUDGET 0eb8fb8 Correct typo in documentation of MBEDTLS_SSL_RENEGOTIATION 76cf60b Remove merge conflict marker in ssl-opt.sh e4f965d Adapt ChangeLog 169712e Merge remote-tracking branch 'restricted/pr/390' into development a74faba aes: xts: Correct NIST 80-38E to 800-38E 8381fcb aes: xts: Update inaccurate comments cc40d86 Improve documentation of mbedtls_ssl_get_verify_result() a966fde Fix some documentation typos and improve a comment c774e32 Merge remote-tracking branch 'restricted/pr/517' into development 837f821 Merge remote-tracking branch 'public/pr/1990' into development ca29fdf Fix some typos in documentation and comments bcedb92 Merge branch 'development' into iotssl-1260-non-blocking-ecc-restricted c028afb Merge pull request openthread#2121 from dgreen-arm/mbedtls-jenkinsfile b51e267 Add Jenkinsfile for PR job f24c336 Adapt ChangeLog dd3ab13 Fail when encountering invalid CBC padding in EtM records 198611d Add missing return value check in ECDSA test suite 96f3b4e Remove yotta support from check-files.py b25cb60 Add a comment to clarify code flow a5a3e40 Fix missing dereference. 7a28e99 Expand test to ensure no assumption on output 90f31b7 Improve readability by moving counter decrement a58e011 Fix alignment in a macro definition b843b15 Fix function name to fit conventions c0edc96 Add comment on internal function API d8b73f2 Remove unnecessary calls to init() from free() c37423f Fix misleading sub-state name and comments 5267b62 Change error description a27190b Rename platform error code and value 25781f9 Fix missing check in example client ee68cff Fix or improve some comments (and whitespace) 32df911 Fix documentation of what functions restart when 7e1f3be Adapt ChangeLog f0bbd7e Misc documentation fixes/improvements. 6346a75 Merge branch 'development' into iotssl-1260-non-blocking-ecc-restricted 0b44d5c Zeroize sensitive data in aescrypt2 and crypt_and_hash examples 805f2e1 Add missing zeroization of buffered handshake messages 9580528 ssl-opt.sh: fix typo in mtu settings b459346 ssl-opt.sh: add a check for i686 targets 7782605 ssl-opt.sh: add comments regarding ciphersuite forcing 748face ssl_tls: fix maximum output length 7c6df83 ssl-opt.sh: revert unnecessary "autoreduction" checks 7311c78 ssl-opt.sh: force ciphersuites to reduce mtu size 780f0a4 Reinitialize PK ctx in mbedtls_pk_parse_key before reuse are free cfa2e33 Omit runtime configuration of calloc/free if macro config enabled e5fecec Add test for MBEDTLS_PLATFORM_{CALLOC/FREE}_MACRO to all.sh 8df1023 Add explicit unsigned-to-signed integer conversion ac2ead0 Improve deterministic test for prime testing e3f95ed Fix bias in random number generation in Miller-Rabin test e0e7ddf Changelog: Add entry for prime validation fix a0b67c2 Bignum: Deprecate mbedtls_mpi_is_prime() da31fa1 Bignum: Fix prime validation vulnerability 64eca05 Bignum: Add tests for primality testing b728c29 Bignum: Remove dead code 3332937 Changelog: Add entry for prime test improvement b8fc1b0 RSA: Use MBEDTLS_MPI_GEN_PRIME_FLAG_LOW_ERR a3cb7eb Bignum: Add test for improved prime generation f301d23 Bignum: Improve primality test for FIPS primes 35f2f30 ssl-opt.sh: change expected pattern from 'resend' to 'autoreduction' ef43ce6 Dtls: change the way unlimited mtu is set for client hello messages abe6003 Adapt ChangeLog 095d9cf Fix ordering of free()ing of internal structures in ssl_server2 0fc9cf4 ssl-opt.sh: increase proxy mtu sizes to contain entire datagrams 1624e2e Avoid overly long lines X.509 DN attr array def in x509_create.c 35b6854 Replace reference to RFC 3280 by reference to newer RFC 5280 cfc47ba Correct some indentation and line lengths in x509_create.c d355e69 Rename `tag` to `default_tag` in x509_attr_descriptor_t d0e21fb Improve documentation of ASN.1 string-writing functions d2c9009 Improve documentation of x509_attr_descriptor_t dc71ef8 Adapt ChangeLog 0c161d1 Fix bounds check in ssl_parse_server_psk_hint() 948fe80 ssl-opt.sh: adjust test timeouts to fit slower targets 6290dae Disable dtls fragmentation for ClientHello messages 52f8491 ssl-opt.sh: adjust tests to fit slower targets 617a321 Adapt ChangeLog 30a9510 Fix memory leak and freeing without initialization in cert_write 16b1bd8 bn_mul.h: add ARM DSP optimized MULADDC code d264258 Make inclusion of stdio.h conditional in x509_crt.c 95c5575 check-files: exclude .git and third-party files 404aa65 Add ChangeLog entry for Windows threading fix 6e3606e Fix run-test-suites.pl to screen for files 427df37 Don't try to disable ASLR 5c39d7a Remove redundant check in all.sh bd90a8c In keep-going mode, don't hard-fail on some tests a9daa5c Look for documentation only in specific directories 65593d2 rsa: pss: Add no possible salt size tests 80d99ea rsa: pss: Extend tests for variable salt length 3725bb2 rsa: pss: Enable use of big hashes with small keys 0592ea7 Merge remote-tracking branch 'public/pr/1140' into development 1afc767 Merge remote-tracking branch 'public/pr/1758' into development c86993e Merge remote-tracking branch 'public/pr/1970' into development c0514bf Added ChangeLog entry for yotta support removal 3ad2efd Remove Yotta support from the docs, tests and build scripts 487cdf5 In keep-going mode, don't hard-fail on some auxiliary script 2561a50 Fix "make WINDOWS_BUILD=1 clean" on non-Windows hosts 7c025a9 Generalize dh_flag in mbedtls_mpi_gen_prime eba6c9b changes requested by @hanno-arm - renamed `mbedtls_asn1_write_any_string` to `mbedtls_asn1_write_tagged_string` - renamed `x509_at_oid_from_name` to `x509_attr_descr_from_name` 54a7c62 Minor style changes 5a481f1 Update error.h count for SSL 12e4a8b Improve documentation wording and formatting 1c1c20e Fix some whitespace issues ff21572 rsa: pss: Use size_t when computing signatures 125af94 Merge branch 'development-restricted' into iotssl-1260-non-blocking-ecc-restricted 53546ea Update library version number to 2.13.1 5d40f67 Merge remote-tracking branch 'public/pr/1927' into development-restricted d2ef254 Don't define _POSIX_C_SOURCE in header file f5106d5 Don't declare and define gmtime()-mutex on Windows platforms 323d801 Correct preprocessor guards determining use of gmtime() 03b2bd4 Correct documentation of mbedtls_platform_gmtime_r() a50fed9 Correct typo in documentation of mbedtls_platform_gmtime_r() 6f70581 Correct POSIX version check to determine presence of gmtime_r() c52ef40 Improve documentation of mbedtls_platform_gmtime_r() 7dd82b4 platform_utils.{c/h} -> platform_util.{c/h} 5a7fe14 Don't include platform_time.h if !MBEDTLS_HAVE_TIME 9fbbf1c Improve wording of documentation of MBEDTLS_PLATFORM_GMTIME_R_ALT c946888 Fix typo in documentation of MBEDTLS_PLATFORM_GMTIME_R_ALT 921b76d Replace 'thread safe' by 'thread-safe' in the documentation 9a51d01 Improve documentation of MBEDTLS_HAVE_TIME_DATE acef292 ChangeLog: Add missing renamings gmtime -> gmtime_r 4e67cca Improve documentation of MBEDTLS_HAVE_TIME_DATE 48a816f Minor documentation improvements 651d586 Style: Add missing period in documentation in threading.h 6a73978 Rename mbedtls_platform_gmtime() to mbedtls_platform_gmtime_r() be2e4bd Guard decl and use of gmtime mutex by HAVE_TIME_DATE and !GMTIME_ALT 5f95c79 Remove another mentioning of IAR from config.h 272675f Correct documentation of mbedtls_platform_gmtime() cfeb70c gmtime: Remove special treatment for IAR 94b540a Avoid redefining _POSIX_C_SOURCE 45e3020 Document that IAR gmtime_s() is auto selected 433f911 Check for IAR in gmtime macros e58088e Clarify docs for MBEDTLS_HAVE_TIME_DATE c29c34c Improve wording of gmtime feature in ChangeLog 3c9733a Fix typo in comment for gmtime macro defines 193fe89 Add missing _POSIX_C_SOURCE define in threading.h ca04a01 Document shorthand gmtime macros 8c9a620 Fix missing word in ChangeLog entry for gmtime() 2099606 Use gmtime_s() for IAR e9b10b2 Define _POSIX_C_SOURCE in threading.c before POSIX detection c2f948b Fix grammar in docs for MBEDTLS_HAVE_TIME_DATE 43e4ff0 Make definitions easier to understand 521e8a9 Fix to comment to correct module description 82986c1 Disable 3d test with no good openssl version c1eda67 Disable fragmentation tests with openssl bugs da642d9 Implicit _endthread call: comment changed e1b82ad Added spaces after type casts acedc91 Fix build issue on clang e46c329 restore missing operation to CMakeLists.txt 85fcb58 Refactor test code for CTR DRBG to clarify test functions b0ad581 Remove the invalid input for platform module 0ff4e0b Add platform error codes 6275be3 Re-factor test functions and add support for data_t parameters b7ebbca compile time warning of 128bit ctr_drbg keys and standardized warnings ce266e4 use single define for 128bit key for ctr_drbg and update test dependencies 6a4f224 ssl-opt.sh: change expected output for large srv packet test with SSLv3 a658d7d Fix style for mbedtls_platform_gmtime() c99b12b Fix documentation for MBEDTLS_HAVE_DATE_TIME 5e863e0 Adapt ChangeLog aa71500 Fix bug in SSL ticket implementation removing keys of age < 1s 1d73993 ssl-opt.sh: Add DTLS session resumption tests aecf8c9 adding more ctr_drbg test vectors (CAVP 14.3) e4b77b9 ctr_drbg: add a few test vectors from CAVP 14.3 cfc1de7 ctr_drbg: sample vectors from NIST 5ef5a9a ctr_drbg tests: unify validate functions 4c78665 ctr_drbg tests: new validation function that doesn't reseed ef0624f ctr_drbg tests: make input sizes flexible ed7da59 ctr_drbg: add comments relating the code with the NIST specification 954f987 change warning about 128bit keys to print eb73f7a fix trailing white spaces 04354b0 add ChangeLog entry c9547cc add NIST test vector for 128 bit keys (not passing yet). 604e60d add default value for CTR DRBG key size a4588d4 enable the use of variable size keys in ctr_drbg (was hard-coded 256bit before) 248e27c Remove redundant statement from x509_get_current_time a7b9f15 Add ChangeLog entry for configurable gmtime() in platform 1abb368 Make gmtime() configurable at compile-time ffbdc61 Add test/zeroize.c 0b54419 More grammar alignment d717743 Fix check-names.sh fail with USE_GMTIME macro 97f3ecb Document dependency on gmtime, gmtime_r & gmtime_s 824dfb3 Add ChangeLog entry for use of gmtime ce6eebb Use gmtime when target is not windows or posix aa22030 Further wording improvements 27a0460 Unify the grammar of descriptions c2e5cdd Explain aescrypt2 better and warn that it doesn't do things properly 6b9cbb8 Copyediting bb4bbbb Resolve PR review comments 6f90ed8 Fix CI failure 4e64e0b Fix after PR comments 7b01244 Add tests for mbedtls_cipher_crypt API 604ccc6 Add ChangeLog entry 636179a Fix typo 278af45 Fix hmac_drbg failure in benchmark, with threading c8c12b6 Add NULL pointer check for consistency 78d7e8c Rename internal variable for consistency 4edb9cb Justify some max_ops settings in test_suite_pk 823c915 Fix mis-merged ChangeLog 597dbf8 Adds a filter to run-test-suites.pl to exclude data files 42c73a9 Merge remote-tracking branch 'origin/development' into development 755890f Remove trailing whitespace e931d0e Replace Windows API threading with CRT functions 5c7e76e Remove trailing whitespace da4029d ssl_server2: add buffer overhead for a termination character 95e2eca Fix IAR warning c19fc55 Add missing large and small packet tests for ssl_server2 30e731d Added buffer_size and response_size options for ssl-server2. Added appropriate tests. 79d9b50 Merge branch 'development' into iotssl-1260-non-blocking-ecc-restricted 558da9c Make SSL error code more generic b5d668a post-merge: adapt ssl-opt.sh to new error values d0a66cc post-merge: fix ECDSA test vector testing da19f4c Merge branch 'development' into iotssl-1260-non-blocking-ecc-restricted 020c823 fixed segmentation fault c150f0d fixed missing initializer c5c9aaf Merge branch 'development' of https://github.com/thomas-dee/mbedtls into development 23f954d Fix string downcast to `PrintableString` as issued in openthread#1033 3efa888 Update ChangeLog with dtls sample IPv6 change 6fd96ad Change the default IP addresses for DTLS samples ceb7b12 Readme with a short description of each sample program a4dd783 Fix some typos in comments bb216bd Fix MSVC 64-bit warnings 196d133 Fix uninitialised variable in some configs 46ba7f3 Avoid running useless code in tests fd87e35 Improve comments on parity trick 95aedfe Remove redundant test on curve type 11556e2 Clarify initialization of T in mul_comb() 0b23f16 SSL: rework restart state handling 6348181 Explicit state assignments in ecdsa too 4ed1dab ECP: move state changes closer to operations c9efa00 ECP: Use explicit state assignments db4a8eb Use ECP_RS_ENTER/LEAVE() macros, as in ECDSA b948f7d Don't bother to free NULL subcontexts bfa1972 Remove redundant checks, save a few muls 28d1628 Avoid unnecessary xxx_copy() calls 5bd38b1 Replace memset() calls with xxx_init() calls 92cceb2 Make some names more consistent ebac5d3 Fix some whitespace & style issues eaf55be Misc documentation fixes/improvements 7037e22 Improve comments and doc for ECP daf0491 Rework state saving for verify_chain() a968843 Improve some comments in verify_chain() 3627a8b Clarify state handling in find_parent(_in)() 83e923b Better initialisation of ver_chain 8b7b96b Fix typo d55f776 Skip context allocation if restart disabled aaa9814 Uniformize ifdefs to ECDSA_C+ECP_RESTARTABLE fe68770 Keep PK layer context in the PK layer 0bbc66c Dynamically allocate/free restart subcontext in PK 15d7df2 Introduce mbedtls_pk_restart_ctx and use it 98a6778 Better document some function arguments b889d3e Clarify & uniformise test comments 5faafa7 Update X.509 test certs' Readme c9e16a9 Disable restartable ECC by default 9897cc9 Update ChangeLog 3bf49c4 Enable restart for certificate verify fed37ed Extract some code to separate function 39eda87 Make more auto variables const 6b7301c Change restart context type. d27d1a5 Clean up existing SSL restartable ECC code 8b59049 Make verify() actually restartable c11e4ba Rework type for verify chain 18547b5 Refactor find_parent() to merge two call sites a4a5d1d Adapt function signatures to rs_ctx + ret be4ff42 Call crt_check_signature from one place only d19a41d Add tests for verify_restartable() bc3f44a Introduce mbedtls_x509_crt_verify_restartable() 21b7719 Add ChangeLog entry for current progress 32033da Test some more handshake flows 23e4162 ECDH: not restartable unless explicitly enabled 1f1f2a1 Adapt ServerKeyEchange processing to restart 862cde5 Add restart support for ECDSA client auth 2350b4e Adapt ECDHE_ECDSA key exchange to restartable EC b3c8307 Adapt ssl_client2 to restartable EC 171a481 Add a ChangeLog entry for changes so far 31f0ef7 Fix style issues introduced earlier 1f59606 Make PK EC sign/verify actually restartable c4ee9ac Add tests for restartable PK sign/verify 82cb27b PK: declare restartable sign/verify functions fd838da Comment cosmetics 66ba48a Make ECDH functions actually restartable 71b2c53 Add tests for restartable ECDH c90d3b0 Update doc for restartable ECDH functions af081f5 Make ECDSA sign actually restartable 50b63ba Use ecp_gen_privkey() in ECDSA sign 6754396 Improve sign/key_tries handling b90883d Prepare infra for restartable sign eb402f3 Add test for restartable signature addb10e Create functions for restartable sign 5314f23 Make verify_restartable() actually restartable a0c5bcc Add infrastructure for ecdsa_verify_restartable() 722e515 Add test for ecdsa_read_signature_restartable() 32aa437 Declare ecdsa_read_signature_restartable() 1ed2505 Add some negative testing for ecdsa_verify() c751148 Make some macros/functions public b5a50e7 Always declare restartable function variants 0ff17df Fix ECDSA test descriptions 4b9c51e Rename EARLY_RETURN -> RESTARTABLE 411079f Rm obsolete comment a7937f9 Add public function generating private keys 1631d63 Make muladd_restartable() actually restartable 54dd652 Introduce muladd_restartable() and its sub-context a08cd1a Avoid re-checking argument on restart 3a25612 Reset ops_done at the right time 53fbd63 Remove redundant check 884569c Avoid code duplication in ecp_mul() 646393b Move ops count to top-level context 8467e68 Stop checking for argument change 3cade22 Switch to restart context internally 8f28add Update documentation for new design/API b739a71 Start moving to new design/API 45fd016 Rename multiplication-specific restart context e58f65a Expand documentation with notes and warnings e685449 Scale ops count for larger curves d3a0ca8 Add tests about number of restarts 9c5c78f Fix indicative values of ops counts 1c678e0 Update doc about minimum max_ops value 213541a Make the first precomp loop restartable ae55707 Turn double loop into single loop e2d7cb3 Start splitting precompute_comb() 67c83fb Give a constant a name fc3e0be Separate auxiliary array in precompute_comb() 07bf6f5 Tune T ownership code + comments 085b1df Allow T to be computed in multiple steps c9c0aa6 Remember precomputed table c5d844b Full restart support in ecp_mul_comb_core() 2fad7ae Start actually splitting computation 8962ddb Don't write to destination until we're done 78d564a Add check for changing arguments 77af79a Add proper allocation of restart context 62738e9 Further restrict variable scope by moving code 391f441 Move more code to separate function 24be795 Group related code together 4b2336d Move some more code to new function 22be635 Re-order some more code ec5606a Extract code to separate function 7306dff Group related code together 510d5ca Add early return test + fake implementation 054433c Add mbedtls_ecp_set_max_ops() 5e3c62f Add MBEDTLS_ERR_ECP_IN_PROGRESS c3a3bc7 Add config flag MBEDTLS_ECP_EARLY_RETURN 225b37a Fix typos in comment git-subtree-dir: third_party/mbedtls/repo git-subtree-split: f00d4bea1709e24c6f407b6aa296e8d283bca684
- Add stdio.h for snprintf. - When `MBEDTLS_ECP_ALT` is defined, need to define mbedtls_ecp_restart_ctx for build to succeed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.