diff options
| author | Markus Uhlin <markus@nifty-networks.net> | 2026-04-27 00:20:41 +0200 |
|---|---|---|
| committer | Markus Uhlin <markus@nifty-networks.net> | 2026-04-27 00:20:41 +0200 |
| commit | 7b9d6a23121bd43936e1bd73095abe9b261e7a37 (patch) | |
| tree | 773c0001991aa4608024ddc5d7c6101c1cf6a4aa /FICS | |
| parent | 126439e49d775999c37c6d8944d627afe37d6d71 (diff) | |
Improved add_handle_to_list()
Diffstat (limited to 'FICS')
| -rw-r--r-- | FICS/fics_addplayer.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/FICS/fics_addplayer.c b/FICS/fics_addplayer.c index 7b6ec30..25ab4f1 100644 --- a/FICS/fics_addplayer.c +++ b/FICS/fics_addplayer.c @@ -44,6 +44,7 @@ #include "config.h" #include "fics_getsalt.h" #include "ficsmain.h" +#include "maxxes-utils.h" #include "playerdb.h" #include "prep_dir_for_privdrop.h" #include "settings.h" @@ -66,11 +67,15 @@ add_handle_to_list(const char *handle) { FILE *fp; char path[1024] = { '\0' }; - int fd; + int fd, ret; - snprintf(path, sizeof path, "%s/admin", DEFAULT_LISTS); + ret = snprintf(path, sizeof path, "%s/admin", DEFAULT_LISTS); - if ((fd = open(path, g_open_flags[OPFL_APPEND], g_open_modes)) < 0) { + if (is_too_long(ret, sizeof path)) { + warnx("%s: error: path too long", __func__); + return; + } else if ((fd = open(path, g_open_flags[OPFL_APPEND], + g_open_modes)) < 0) { warn("%s: unable to open %s", __func__, path); return; } else if ((fp = fdopen(fd, "a")) == NULL) { @@ -79,8 +84,10 @@ add_handle_to_list(const char *handle) return; } - fprintf(fp, "%s\n", handle); - fclose(fp); + mfprintf(fp, "%s\n", handle); + + if (fclose(fp) != 0) + warn("%s: error closing file pointer", __func__); } PRIVATE __dead void |