Skip to content

Fix possible stack overflow#134

Closed
0xPxt wants to merge 4 commits intomainfrom
hotfix/memcpy_overflow
Closed

Fix possible stack overflow#134
0xPxt wants to merge 4 commits intomainfrom
hotfix/memcpy_overflow

Conversation

@0xPxt
Copy link
Collaborator

@0xPxt 0xPxt commented May 28, 2025 •
edited
Loading

  • Added a boundary check before memcpy
  • Rust CI

windsurf-bot[bot]
windsurf-bot bot reviewed May 28, 2025
View reviewed changes
Copy link

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

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

1 file skipped due to size limits:
  • app/src/parser_print_txn.c

@0xPxt 0xPxt force-pushed the hotfix/memcpy_overflow branch 11 times, most recently from e88d604 to fa822e8 Compare May 28, 2025 12:55
@0xPxt 0xPxt force-pushed the hotfix/memcpy_overflow branch from fa822e8 to fc3c108 Compare May 28, 2025 12:59
@chcmedeiros chcmedeiros self-requested a review May 28, 2025 13:54
@0xPxt 0xPxt closed this May 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chcmedeiros chcmedeiros chcmedeiros approved these changes

+1 more reviewer

@windsurf-bot windsurf-bot[bot] windsurf-bot[bot] left review comments

Reviewers whose approvals may not affect merge requirements

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

@0xPxt @chcmedeiros