diff options
| author | Markus Uhlin <markus@nifty-networks.net> | 2025-03-09 17:17:01 +0100 |
|---|---|---|
| committer | Markus Uhlin <markus@nifty-networks.net> | 2025-03-09 17:17:01 +0100 |
| commit | a2aa16d0baef72700fc14a348abace5560a4d7ab (patch) | |
| tree | 6363f9c646efbc389205ecbfbe44459ec9b7a8c3 | |
| parent | fef073b47182e14bc8822be4699ec5a1ff3bdeba (diff) | |
create_news_file: fixed null ptr dereferences
| -rw-r--r-- | FICS/adminproc.c | 14 |
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); + } } } |