Skip to content

[BUG] console: Fix buffer overflow#795

Merged
minminlittleshrimp merged 1 commit intomasterfrom
ShangzhiXu/master
Dec 23, 2025
Merged

[BUG] console: Fix buffer overflow#795
minminlittleshrimp merged 1 commit intomasterfrom
ShangzhiXu/master

Conversation

@minminlittleshrimp
Copy link
Collaborator

Fix dlt sortbytimestamp buffer Overflow by:

  • Avoid set num = end+1 (out of range)
  • Avoid using num as index in case of entending

Full report and demo can be checked in Issue 792.
Reported by Shangzhi-Xu.

@minminlittleshrimp
Copy link
Collaborator Author

@copilot

Copy link
Contributor

Copilot AI commented Dec 23, 2025

@minminlittleshrimp I've opened a new pull request, #796, to work on those changes. Once the pull request is ready, I'll request review from you.

Fix dlt sortbytimestamp buffer Overflow by:
- Avoid set num = end+1 (out of range)
- Avoid using num as index in case of entending

Full report and demo can be checked in Issue 792.
Reported by Shangzhi-Xu.

Signed-off-by: Shangzhi-Xu <63028857+ShangzhiXu@users.noreply.github.com>
Signed-off-by: LUU QUANG MINH <Minh.LuuQuang@vn.bosch.com>
Copy link
Collaborator

@lti9hc lti9hc left a comment

Choose a reason for hiding this comment

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

it fixed now
thanks for your fix
@minminlittleshrimp

@minminlittleshrimp minminlittleshrimp merged commit e885cff into master Dec 23, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Protential heap-buffer-overflow crash in dlt-sortbytimestamp.c

4 participants