Skip to content

Fix unbounded writing of sscanf#55

Open
szsam wants to merge 1 commit intosyoyo:masterfrom
szsam:patch-1
Open

Fix unbounded writing of sscanf#55
szsam wants to merge 1 commit intosyoyo:masterfrom
szsam:patch-1

Conversation

@szsam
Copy link
Contributor

@szsam szsam commented Apr 22, 2023

'%s' may lead to buffer overflow if the width is not provided.

'%s' may lead to buffer overflow if the width is not provided.
@syoyo
Copy link
Owner

syoyo commented Apr 24, 2023

sscanf in this case is simply for extracting a name string from a char array.
It is better to implement our own sscanf alternative(strip whitespaces and extract a string)

I'll try to implement it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants