Skip to content

Various fixes - #21

Merged
agrojean-ledger merged 4 commits into
developfrom
increase-allowed-node-children
Oct 17, 2025
Merged

Various fixes#21
agrojean-ledger merged 4 commits into
developfrom
increase-allowed-node-children

Conversation

@agrojean-ledger

@agrojean-ledger agrojean-ledger commented Oct 17, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator
  • Allow up to 16 node children during prepared transaction parsing (for hashing).
  • Quick fix for memory leaks between transactions : reset memory allocation buffer when receiving first APDU. This should be replaced by clean memory management.

Checklist

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

Fingolfin69
Fingolfin69 approved these changes Oct 17, 2025
View reviewed changes

@Fingolfin69 Fingolfin69 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GJ 👍

@agrojean-ledger
agrojean-ledger merged commit b81fd36 into develop Oct 17, 2025
47 checks passed
@agrojean-ledger
agrojean-ledger deleted the increase-allowed-node-children branch October 17, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Fingolfin69 Fingolfin69 Fingolfin69 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@agrojean-ledger @Fingolfin69