Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Replaced sprintf with safer snprintf #22149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Jun 25, 2022
Merged

Replaced sprintf with safer snprintf #22149

merged 2 commits into from
Jun 25, 2022

Conversation

seanm
Copy link
Contributor

@seanm seanm commented Jun 22, 2022 •
edited by alalek
Loading 

force_builders=Custom
build_image:Custom=centos:7
buildworker:Custom=linux-f1

seanm added 2 commits June 22, 2022 12:46
@seanm
Straightforward replacement of sprintf with safer snprintf
666f0e5
@seanm
Trickier replacement of sprintf with safer snprintf
808acc0 
Some functions were changed to take another parameter: the size of the buffer, so that they can pass that size on to snprintf.
@seanm seanm changed the title Sprintf Replaced sprintf with safer snprintf Jun 22, 2022
alalek
alalek approved these changes Jun 23, 2022
View reviewed changes
Copy link
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@alalek alalek merged commit 35f1a90 into opencv:4.x Jun 25, 2022
@alalek alalek mentioned this pull request Aug 21, 2022
Merged
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
@seanm
Merge pull request opencv#22149 from seanm:sprintf
b787eaf 
Replaced sprintf with safer snprintf

* Straightforward replacement of sprintf with safer snprintf

* Trickier replacement of sprintf with safer snprintf

Some functions were changed to take another parameter: the size of the buffer, so that they can pass that size on to snprintf.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alalek alalek alalek approved these changes

Assignees

@alalek alalek

Labels

None yet

Projects

None yet

Milestone

4.7.0

Development

Successfully merging this pull request may close these issues.

2 participants

@seanm @alalek