diff options
| author | Markus Uhlin <markus@nifty-networks.net> | 2024-03-16 14:57:50 +0100 |
|---|---|---|
| committer | Markus Uhlin <markus@nifty-networks.net> | 2024-03-16 14:57:50 +0100 |
| commit | 7b43af3149b3cd816f1674a2096c8f71a157d920 (patch) | |
| tree | 9e5f355b10e483e01f0d4bcdaf6321bfd4964156 /FICS/utils.c | |
| parent | 2a3e9e2f0880068c1f5e13585b4854d29c5865bb (diff) | |
Replaced sprintf() calls with snprintf() and strcpy() calls with strlcpy()
Diffstat (limited to 'FICS/utils.c')
| -rw-r--r-- | FICS/utils.c | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/FICS/utils.c b/FICS/utils.c index f842a8b..c7d6774 100644 --- a/FICS/utils.c +++ b/FICS/utils.c @@ -35,6 +35,10 @@ #include "rmalloc.h" #include "utils.h" +#if __linux__ +#include <bsd/string.h> +#endif + struct t_tree { struct t_tree *left, *right; char name; @@ -378,9 +382,9 @@ psend_raw_file(int p, char *dir, char *file) int num; if (dir) - sprintf(fname, "%s/%s", dir, file); + snprintf(fname, sizeof fname, "%s/%s", dir, file); else - strcpy(fname, file); + strlcpy(fname, file, sizeof fname); if ((fp = fopen(fname, "r")) == NULL) return -1; @@ -408,9 +412,9 @@ psend_file(int p, char *dir, char *file) parray[p].last_file_byte = 0L; if (dir) - sprintf(fname, "%s/%s", dir, file); + snprintf(fname, sizeof fname, "%s/%s", dir, file); else - strcpy(fname, file); + strlcpy(fname, file, sizeof fname); if ((fp = fopen(fname, "r")) == NULL) return -1; @@ -449,9 +453,9 @@ psend_logoutfile(int p, char *dir, char *file) parray[p].last_file_byte = 0L; if (dir) - sprintf(fname, "%s/%s", dir, file); + snprintf(fname, sizeof fname, "%s/%s", dir, file); else - strcpy(fname, file); + strlcpy(fname, file, sizeof fname); if ((fp = fopen(fname, "r")) == NULL) return -1; @@ -840,7 +844,7 @@ file_wplayer(char *fname) char *ptr; static char tmp[MAX_FILENAME_SIZE]; - strcpy(tmp, fname); + strlcpy(tmp, fname, sizeof tmp); if ((ptr = rindex(tmp, '-')) == NULL) return ""; |