Skip to content

Fix buffer overflow in engine_status_dump() function#315

Open
dmitrym1 wants to merge 1 commit intoXilinx:masterfrom
dmitrym1:fix_engine_status_dump_buffer_overflow
Open

Fix buffer overflow in engine_status_dump() function#315
dmitrym1 wants to merge 1 commit intoXilinx:masterfrom
dmitrym1:fix_engine_status_dump_buffer_overflow

Conversation

@dmitrym1
Copy link

fixes #314, also fixes a typo (INVLIAD_LEN => INVALID_LEN)

@Prandr
Copy link

Prandr commented Apr 23, 2025

@dmitrym1
Hi,
I think this all is really unnecessary. If you would like to improve the function, get rid of the messy buffer and sprintf altogether and switch to use pr_cont.

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.

engine_status_dump buffer overflow

2 participants