Skip to content

Replace sprintf() with snprintf() when available#29

Merged
attipaci merged 4 commits into
mainfrom
snprintf
May 21, 2026
Merged

Replace sprintf() with snprintf() when available#29
attipaci merged 4 commits into
mainfrom
snprintf

Conversation

@attipaci

@attipaci attipaci commented May 20, 2026

Copy link
Copy Markdown
Collaborator

Edit also change string length parameters from int to size_t or long (if it needs to be signed)

@attipaci attipaci added this to the 1.2.0 milestone May 20, 2026
@attipaci attipaci self-assigned this May 20, 2026
@attipaci attipaci added the enhancement New feature or request label May 20, 2026
Comment thread src/xchange.c Fixed
Comment thread src/xjson.c Fixed
Comment thread src/xjson.c Fixed
@attipaci attipaci force-pushed the snprintf branch 3 times, most recently from 364c99f to 2e0be4b Compare May 20, 2026 13:34
Comment thread src/xchange.c Fixed
Comment thread src/xchange.c Fixed
@attipaci attipaci force-pushed the snprintf branch 4 times, most recently from b25fa72 to 538aba2 Compare May 20, 2026 14:08
@attipaci attipaci marked this pull request as ready for review May 20, 2026 14:12
@attipaci attipaci force-pushed the main branch 4 times, most recently from c8c48ba to 78af5f2 Compare May 20, 2026 19:01
@attipaci attipaci force-pushed the snprintf branch 2 times, most recently from 4bcf6ae to eb0742a Compare May 21, 2026 07:52
Comment thread src/xchange.c Fixed
Comment thread src/xjson.c Fixed
Comment thread src/xjson.c Fixed
@attipaci attipaci force-pushed the snprintf branch 3 times, most recently from 9e72fe4 to fd8459a Compare May 21, 2026 08:32
Comment thread src/xjson.c Fixed
@attipaci attipaci force-pushed the snprintf branch 2 times, most recently from 73ae478 to bdcadba Compare May 21, 2026 08:39
@attipaci attipaci force-pushed the snprintf branch 8 times, most recently from 8bf5721 to 79fb259 Compare May 21, 2026 11:07
@attipaci attipaci changed the title Replace sprintf() with snprintf() everywhere... Replace sprintf() with snprintf() when available May 21, 2026
@attipaci attipaci merged commit 9e59c49 into main May 21, 2026
24 checks passed
@attipaci attipaci deleted the snprintf branch May 21, 2026 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants