Skip to content

GameSpy: Fix the warning about sprintf being insecure.#672

Merged
Lgt2x merged 1 commit intoDescentDevelopers:mainfrom
MaddTheSane:gs-printfFix
Feb 27, 2025
Merged

GameSpy: Fix the warning about sprintf being insecure.#672
Lgt2x merged 1 commit intoDescentDevelopers:mainfrom
MaddTheSane:gs-printfFix

Conversation

@MaddTheSane
Copy link
Contributor

@MaddTheSane MaddTheSane commented Feb 27, 2025

Pull Request Type

  • GitHub Workflow changes
  • Documentation or Wiki changes
  • Build and Dependency changes
  • Runtime changes
    • Render changes
    • Audio changes
    • Input changes
    • Network changes
    • Other changes

Description

This replaces the calls to sprintf with snprintf where easy in the GameSpy code.

Checklist

  • I have tested my changes locally and verified that they work as intended.
  • I have documented any new or modified functionality.
  • I have reviewed the changes to ensure they do not introduce any unnecessary complexity or duplicate code.
  • I understand that by submitting this pull request, I am agreeing to license my contributions under the project's license.

Copy link
Member

@Lgt2x Lgt2x left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@Lgt2x Lgt2x merged commit efd3e8e into DescentDevelopers:main Feb 27, 2025
10 checks passed
@MaddTheSane MaddTheSane deleted the gs-printfFix branch February 27, 2025 09:29
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.

2 participants