aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Uhlin <markus@nifty-networks.net>2026-03-14 22:35:52 +0100
committerMarkus Uhlin <markus@nifty-networks.net>2026-03-14 22:35:52 +0100
commit7feae69a436df186dc43554c47a579c44e690ecf (patch)
tree6c3d831b96ef8dcba92f536bf38630c92d7a526e
parentc780ec5554e1b17233addf40fcd81354a26dee8e (diff)
player_add_message: added error handling
-rw-r--r--FICS/playerdb.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/FICS/playerdb.c b/FICS/playerdb.c
index 217af28..f3213e5 100644
--- a/FICS/playerdb.c
+++ b/FICS/playerdb.c
@@ -2710,6 +2710,7 @@ player_add_message(int top, int fromp, char *message)
char messbody[1024] = { '\0' };
char subj[256] = { '\0' };
int fd;
+ int ret[2];
time_t t = time(NULL);
if (!parray[top].registered)
@@ -2729,9 +2730,14 @@ player_add_message(int top, int fromp, char *message)
return -1;
}
- fprintf(fp, "%s at %s: %s\n", parray[fromp].name, strltime(&t),
+ ret[0] = fprintf(fp, "%s at %s: %s\n", parray[fromp].name, strltime(&t),
message);
- fclose(fp);
+ ret[1] = fclose(fp);
+
+ if (ret[0] < 0 || ret[1] != 0) {
+ warnx("%s: certain file operations failed", __func__);
+ return -1;
+ }
pprintf(fromp, "\nThe following message was sent ");