Skip to content

Conversation

liruqi
Copy link

@liruqi liruqi commented Jan 9, 2023

No description provided.

@cmb69
Copy link
Member

cmb69 commented Jan 9, 2023

Thank you for the PR! Simply allowing too large values would not work, since the functions which accept these parameters are expecting ints. In the worst case, that might result in security issues. Consider to provide a PR to upstream which would change the respective int parameters to size_t. I assume that would open up a can of worms, though.

@cmb69 cmb69 closed this Jan 9, 2023
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