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

Merge on Master #65

Merged
merged 99 commits into from
Apr 30, 2024
Merged

Merge on Master #65

merged 99 commits into from
Apr 30, 2024

Conversation

cedelavergne-ledger
Copy link

https://ledgerhq.atlassian.net/browse/B2CA-1491

Checklist

  • App update process has been followed
  • Target branch is develop
  • Application version has been bumped

mehcode and others added 30 commits March 4, 2022 21:36
@mehcode
tweak code in sign_transaction.c to fix warnings from scan-build
8734f24
@mehcode
chore: add protobufs for HTS, remove pb.c,.h files from repo
deb6589 
Signed-off-by: Ryan Leckey <[email protected]>
@mehcode
add new hedera_format_amount procedure to handle arbitrary decimal sizes
c8dffd6
@mehcode
use nothrow versions of the crypto functions
1d9a5ad
@mehcode
Use explicit_bzero over memset
b93ab13
@mehcode
add support for token transfers and token associate transaction
cf2dc12
@mehcode
feat: add support for TokenMint and TokenBurn
519a5e0 
Signed-off-by: Ryan Leckey <[email protected]>
@lpascal-ledger
Merge pull request #17 from mehcode/lb-hts
71c8f62 
Add support for HTS to Ledger BOLOS App for Hedera
@lpascal-ledger
[add] CI trigger on develop (PR included) + NanoX build
e1e7718
@lpascal-ledger
[add] CI Application compilation with NanoS+ SDK
99cc7b2
@mehcode
fix: use .transfers for TokenTransferList and .value when accessing e…
f59a470 
…xpectedDecimals

Signed-off-by: Ryan Leckey <[email protected]>
@lpascal-ledger
Merge pull request #19 from launchbadge/rl-compile-fix
ee109e6 
fix: use .transfers for TokenTransferList and .value when accessing expectedDecimals
@lpascal-ledger
[bump] v1.1.0
8472d3c
@lpascal-ledger
Merge pull request #18 from LedgerHQ/ci
e998959 
Adding CI and fixing NanoX compilation
@mehcode
feat: add unit tests to prove operation of hedera_format_x
8202434 
Signed-off-by: Ryan Leckey <[email protected]>
@mehcode
fix: hedera_format_x is off by one, decrement reverse index after ini…
6b2aab5 
…tial operation

Signed-off-by: Ryan Leckey <[email protected]>
@mehcode
fix: re-add import for string.h to hedera.c
1265972 
Signed-off-by: Ryan Leckey <[email protected]>
@mehcode
fix: use regular memset for clearing amount format buffer
6802fd3 
Signed-off-by: Ryan Leckey <[email protected]>
@fbeutin-ledger
Merge pull request #20 from launchbadge/rl-fix-format
e0a27ff 
feat: add unit tests to prove operation of hedera_format_x
@fbeutin-ledger
Bump version to 1.1.1
6e3486f
@mehcode
fix: sanitize inputs for hedera_format_amount
c473ab3
@mehcode
fix: validate decimal values and reject >= 20
0059c05
@mehcode
fix: trim trailing zeros in hedera_format
bca4024
@lpascal-ledger
Merge pull request #22 from launchbadge/lb-format-safe
71e2c2b 
fix: sanitize inputs for hedera_format_amount
@lpascal-ledger
[Bump] Version 1.1.2
f7e6d70
@lpascal-ledger
Merge pull request #24 from LedgerHQ/bump
2b77a8c 
[Bump] Version 1.1.2
@fbeutin-ledger
Remove trailing spaces
6bd97ed
@fbeutin-ledger
Enable CodeQL analysis
f19805f
@fbeutin-ledger
Fix display of TokenAssociate flow for LNX and LNS+
89a00de
@fbeutin-ledger
Add TokenMint and TokenBurn flows to the LNX and LNS+ devices
c020c4f
fbeutin-ledger and others added 28 commits April 12, 2023 15:47
@fbeutin-ledger
Bump version to 1.3.0
4833576
@fbeutin-ledger
Update snapshots for latest sdk, fonts have changed
be7ff9f
@fbeutin-ledger
Merge pull request #53 from LedgerHQ/fbe/update_snapshots_for_latest_sdk
bab313b 
Update snapshots for latest sdk, fonts have changed
@fbeutin-ledger
Improve stax design
fe3674f
@fbeutin-ledger
Bump version
f3f44f4
@fbeutin-ledger
Merge pull request #54 from LedgerHQ/fbe/improve_stax_UI
da7bd2e 
Improve stax design
@apaillier-ledger
QR code source path is now used explicitly
66d6812
@apaillier-ledger
Merge pull request #55 from LedgerHQ/fix/apa/explicit_qrcode_source_path
d71e6c2 
QR code source path is now used explicitly
@fbeutin-ledger
Use new 4bpp icon
74a1410
@fbeutin-ledger
Merge pull request #56 from LedgerHQ/fbe/4bpp_icon
3f90b67 
Use new 4bpp icon
@Sheng-Long
update the test section in the readme
cc3985e
@Sheng-Long
add section for Ledger's VSCode extension
6dfdbbe
@tdejoigny-ledger
Merge pull request #57 from launchbadge/updateReadmeTest
d974bdc 
update the test section in the readme
@sgliner-ledger
[auto]: add PR template
cde593b
@Sheng-Long
fix: audit code issues
9756908
@Sheng-Long
fix: suggested fixes from ledger
ecd7a38
@Sheng-Long
fix: change appflag to correct value
34d1da7
@Sheng-Long
fix: add manifest
2036392
@xchapron-ledger
Merge pull request #58 from launchbadge/auditFixes
c2827d1 
fix: audit code issues
Bump version to 1.4.1
92961ec
@xchapron-ledger
Merge pull request #60 from LedgerHQ/xch/bump-version
fd3d9c2 
Bump version
[auto] Update screenshot
39add05
@sgliner-ledger
Merge pull request #61 from LedgerHQ/auto-update-screenshots
13fba6e 
[auto] Update Screenshots
[auto] Update screenshot
a29e1cb
@sgliner-ledger
Merge pull request #62 from LedgerHQ/auto-update-screenshots
5c23b17 
[auto] Update Screenshots
@sgliner-ledger
[auto] Update screenshot
d3b7439
@Sheng-Long
fix: address buffer overflows in fuzzer
36aacca
@cedelavergne-ledger
Merge pull request #63 from launchbadge/auditFixes
ae7aaf4 
fix: address buffer overflows in fuzzer
github-advanced-security[bot]
github-advanced-security bot found potential problems Apr 30, 2024
View reviewed changes
src/ui/ui_idle.c Dismissed Show dismissed Hide dismissed
src/hedera_format.c
} else if (buf[ j ] == '.') {
break;
} else {
j += 1;

Check notice

Code scanning / CodeQL

For loop variable changed in body Note

Loop counters should not be modified in the body of the
loop
Loading
.
src/printf.c Dismissed Show dismissed Hide dismissed
src/ui/ui_sign_transaction.c Dismissed Show dismissed Hide dismissed
src/ui/ui_sign_transaction.c Dismissed Show dismissed Hide dismissed
src/ui/ui_sign_transaction.c Dismissed Show dismissed Hide dismissed
@cedelavergne-ledger cedelavergne-ledger merged commit f80a533 into master Apr 30, 2024
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@cedelavergne-ledger @mehcode @lpascal-ledger @fbeutin-ledger @cacampbell @Sheng-Long @apaillier-ledger @tdejoigny-ledger @sgliner-ledger @xchapron-ledger