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

Security Audit passed #5

Merged
merged 49 commits into from
Mar 20, 2024
Merged

Security Audit passed #5

merged 49 commits into from
Mar 20, 2024

Conversation

ajinkyaraj-23
Copy link

Following features are added:

  1. Upgrade to latest ledger-sdk docker images
  2. Fix splash screen for expert mode on stax
  3. Fix security audit comments.
  4. Add code documentation
  5. Upgrade tezos protocol to Oxford (latest)

ajinkyaraj-23 and others added 30 commits January 24, 2024 14:45
…isplay-of-expert-mode-enabled

Change "Expert Mode Enabled" screen to match "TRANSACTION SIGNED" format
…dit-tests

Tests: fix failing tests at app-builder version 3.15.0
[misra-c2012-8.2] Function types shall be in prototype form with named parameters
[misra-c2012-7.2] A "u" or "U" suffix shall be applied to all integer constants that are represented in an unsigned type
[misra-c2012-13.4] The result of an assignment operator should not be used
[misra-c2012-15.6] The body of an iteration-statement or a selection-statement shall be a compound-statement
[misra-c2012-12.1] The precedence of operators within expressions should be made explicit
to prevent it from being included twice
…yle-weaknesses

Style: handle code style weaknesses
APPLICATION_FLAG_GLOBAL_PIN, APPLICATION_FLAG_BOLOS_SETTINGS
…d-ledger-flags

Remove unused Ledger flags
…for-new-entrypoints-introduced-in-oxford-2

Add new Oxford2 entrypoints
…-if-we-dont-return-the-hash

Address audit report comments
…sts-at-latest-app-builder-version

Test: fix failing tests at latest app builder version
Co-authored-by: Ajinkya Rajandekar <[email protected]>
if items other than the first are defined
@ajinkyaraj-23 ajinkyaraj-23 changed the base branch from main to develop February 21, 2024 08:27
ajinkyaraj-23 and others added 4 commits March 15, 2024 12:45
- script to run tests needs to be changed due to updated docker container
- change readme to reflect new way of running the tests.
…n-enabled-splash-screen

Fix blindsign enabled splash screen
@fbeutin-ledger fbeutin-ledger merged commit c75553d into LedgerHQ:develop Mar 20, 2024
104 of 105 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants