aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Uhlin <markus@nifty-networks.net>2025-03-09 17:17:01 +0100
committerMarkus Uhlin <markus@nifty-networks.net>2025-03-09 17:17:01 +0100
commita2aa16d0baef72700fc14a348abace5560a4d7ab (patch)
tree6363f9c646efbc389205ecbfbe44459ec9b7a8c3
parentfef073b47182e14bc8822be4699ec5a1ff3bdeba (diff)
create_news_file: fixed null ptr dereferences
-rw-r--r--FICS/adminproc.c14
1 files changed, 8 insertions, 6 deletions
diff --git a/FICS/adminproc.c b/FICS/adminproc.c
index addf5be..b2859c6 100644
--- a/FICS/adminproc.c
+++ b/FICS/adminproc.c
@@ -183,9 +183,10 @@ create_news_file(int p, param_list param, int admin)
msnprintf(filename, sizeof filename, "%s/adminnews.%d",
news_dir,
param[0].val.integer);
- fp = fopen(filename, "w");
- fprintf(fp, "%s\n", param[1].val.string);
- fclose(fp);
+ if ((fp = fopen(filename, "w")) != NULL) {
+ fprintf(fp, "%s\n", param[1].val.string);
+ fclose(fp);
+ }
}
} else {
if (param[0].val.integer > num_news) {
@@ -195,9 +196,10 @@ create_news_file(int p, param_list param, int admin)
msnprintf(filename, sizeof filename, "%s/news.%d",
news_dir,
param[0].val.integer);
- fp = fopen(filename, "w");
- fprintf(fp, "%s\n", param[1].val.string);
- fclose(fp);
+ if ((fp = fopen(filename, "w")) != NULL) {
+ fprintf(fp, "%s\n", param[1].val.string);
+ fclose(fp);
+ }
}
}