diff options
| author |  Simon Ser <contact@emersion.fr> | 2022-02-08 09:52:24 +0100 |
|---|---|---|
| committer |  Simon Zeni <simon@bl4ckb0ne.ca> | 2022-02-08 09:20:13 -0500 |
| commit | f707f583e17cb5e8323ceb4bfd951ad0465b7d10 (patch) | |
| tree | 9de2eaecf63cb4f8e1c0af7ba1c7c9d7a8fd01a4 /swaynag/config.c | |
| parent | Fix snprintf compiler warning (diff) | |
| download | sway-f707f583e17cb5e8323ceb4bfd951ad0465b7d10.tar.gz sway-f707f583e17cb5e8323ceb4bfd951ad0465b7d10.tar.zst sway-f707f583e17cb5e8323ceb4bfd951ad0465b7d10.zip | |
Remove all sprintf calls
Replace them with snprintf, which ensures buffer overflows won't
happen.
Diffstat (limited to 'swaynag/config.c')
| -rw-r--r-- | swaynag/config.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/swaynag/config.c b/swaynag/config.c index 6db7cce5..ede0938c 100644 --- a/swaynag/config.c +++ b/swaynag/config.c | |||
| @@ -414,8 +414,8 @@ int swaynag_load_config(char *path, struct swaynag *swaynag, list_t *types) { | |||
| 414 | } | 414 | } |
| 415 | free(name); | 415 | free(name); |
| 416 | } else { | 416 | } else { |
| 417 | char *flag = malloc(sizeof(char) * (nread + 3)); | 417 | char *flag = malloc(nread + 3); |
| 418 | sprintf(flag, "--%s", line); | 418 | snprintf(flag, nread + 3, "--%s", line); |
| 419 | char *argv[] = {"swaynag", flag}; | 419 | char *argv[] = {"swaynag", flag}; |
| 420 | result = swaynag_parse_options(2, argv, swaynag, types, type, | 420 | result = swaynag_parse_options(2, argv, swaynag, types, type, |
| 421 | NULL, NULL); | 421 | NULL, NULL); |