Skip to content

Replace sprintf with snprintf to eliminate compile warnings. v6.0.45 #3534

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
winlinvip merged 5 commits into from
May 14, 2023
Merged

Replace sprintf with snprintf to eliminate compile warnings. v6.0.45 #3534

winlinvip merged 5 commits into from
May 14, 2023

Conversation

@chundonglinlin
Copy link
Member

@chundonglinlin chundonglinlin commented May 7, 2023 •
edited by winlinvip
Loading

  • Replaced all occurrences of sprintf with snprintf to address deprecation warnings
  • Ensured proper buffer size is passed to snprintf to prevent potential buffer overflows
  • Ran tests to confirm that the changes do not introduce any new issues or regressions

Remove compile warnings by using snprintf instead of sprintf, likely

warning: 'sprintf' is deprecated: This function is provided for compatibility reasons only.  
Due to security concerns inherent in the design of sprintf(3), it is highly recommended that you use snprintf(3) instead. [-Wdeprecated-declarations]
sprintf(error, "%d:%d: Unknown value", line_and_col);

winlinvip
winlinvip reviewed May 13, 2023
View reviewed changes
winlinvip
winlinvip reviewed May 13, 2023
View reviewed changes
@winlinvip winlinvip changed the title Compile: Use snprintf instead of sprintf. Replace sprintf with snprintf to eliminate compile warnings May 14, 2023
@winlinvip winlinvip changed the title Replace sprintf with snprintf to eliminate compile warnings Replace sprintf with snprintf to eliminate compile warnings. v6.0.45 May 14, 2023
@winlinvip winlinvip merged commit c0e931a into ossrs:develop May 14, 2023
johzzy pushed a commit to johzzy/srs that referenced this pull request Jun 26, 2023
@chundonglinlin @chen-guanghua @johzzy
Replace sprintf with snprintf to eliminate compile warnings. v6.0.45 (o…
6ac7a56 
…ssrs#3534)

* Replaced all occurrences of sprintf with snprintf to address deprecation warnings
* Ensured proper buffer size is passed to snprintf to prevent potential buffer overflows
* Ran tests to confirm that the changes do not introduce any new issues or regressions

---------

Co-authored-by: ChenGH <chengh_math@126.com>
@winlinvip winlinvip added the EnglishNative This issue is conveyed exclusively in English. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@winlinvip winlinvip winlinvip left review comments

@chen-guanghua chen-guanghua chen-guanghua approved these changes

@xiaozhihong xiaozhihong Awaiting requested review from xiaozhihong

Assignees

No one assigned

Labels

EnglishNative This issue is conveyed exclusively in English.

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chundonglinlin @winlinvip @chen-guanghua