Skip to content

Change sprintf to snprintf to eliminate security warnings on MacOS#3498

Merged
kpumuk merged 1 commit into
apache:masterfrom
abellgithub:security-warnings
May 21, 2026
Merged

Change sprintf to snprintf to eliminate security warnings on MacOS#3498
kpumuk merged 1 commit into
apache:masterfrom
abellgithub:security-warnings

Conversation

@abellgithub
Copy link
Copy Markdown
Contributor

…on OSX

Client: cpp

  • Did you create an Apache Jira ticket? (Request account here, not required for trivial changes)
  • If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
  • Did you squash your changes to a single commit? (not required, but preferred)
  • Did you do your best to avoid breaking changes? If one was needed, did you label the Jira ticket with "Breaking-Change"?
  • If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

@mergeable mergeable Bot added the c++ Pull requests that update C++ code label May 19, 2026
@kpumuk kpumuk changed the title No ticket: change sprintf to snprintf to eliminate security warnings … Change sprintf to snprintf to eliminate security warnings on MacOS May 19, 2026
@kpumuk kpumuk merged commit 5748bbb into apache:master May 21, 2026
180 of 183 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Pull requests that update C++ code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants