-
Notifications
You must be signed in to change notification settings - Fork 17
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
Move "easy" basic checks scripts to the framework #95
base: main
Are you sure you want to change the base?
Conversation
ba96d78
to
31423be
Compare
Signed-off-by: Elena Uziunaite <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
Signed-off-by: Elena Uziunaite <[email protected]>
Looks good, however given this #80 (comment) might need to move files from |
31423be
to
fb36949
Compare
Signed-off-by: Elena Uziunaite <[email protected]>
fb36949
to
8b06ebb
Compare
Move scripts used by all-core.sh to the framework
8b06ebb
to
279f311
Compare
The following files are added (imported) from the main Mbed TLS repo: scripts/assemble_changelog.py scripts/check-doxy-blocks.pl scripts/check-python-files.sh scripts/doxygen.sh scripts/apidoc_full.sh scripts/recursion.pl Signed-off-by: Valerio Setti <[email protected]>
279f311
to
0fed0d6
Compare
scripts/recursion.pl
Outdated
# When the recursion depth might depend on data controlled by the attacker in | ||
# an unbounded way, those functions should use iteration instead. | ||
# | ||
# Typical usage: scripts/recursion.pl library/*.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't this be
# Typical usage: scripts/recursion.pl library/*.c | |
# Typical usage: framework/scripts/recursion.pl library/*.c |
now that it's moved?
Signed-off-by: Valerio Setti <[email protected]>
0fed0d6
to
6c7796f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Resolves #69.
development
PR: Mbed-TLS/mbedtls#9825mbedtls-3.6
PR: Mbed-TLS/mbedtls#9826