--- lbbs/src/article_cache.c 2025/06/03 03:14:51 1.12 +++ lbbs/src/article_cache.c 2025/06/16 01:36:56 1.17 @@ -60,6 +60,8 @@ inline static int article_cache_path(cha int article_cache_generate(const char *cache_dir, const ARTICLE *p_article, const SECTION_LIST *p_section, const char *content, const char *sub_ip, int overwrite) { + static char *content_f; // static buffer in large size + char data_file[FILE_PATH_LEN]; int fd; ARTICLE_CACHE cache; @@ -68,7 +70,6 @@ int article_cache_generate(const char *c char header[ARTICLE_HEADER_MAX_LEN]; size_t header_len; long header_line_cnt; - char content_f[ARTICLE_CONTENT_MAX_LEN]; char footer[ARTICLE_FOOTER_MAX_LEN]; size_t footer_len; long footer_line_cnt; @@ -80,6 +81,12 @@ int article_cache_generate(const char *c return -1; } + if (content_f == NULL && (content_f = malloc(ARTICLE_CONTENT_MAX_LEN)) == NULL) + { + log_error("malloc(content_f) error: OOM\n"); + return -1; + } + if (article_cache_path(data_file, sizeof(data_file), cache_dir, p_article) < 0) { log_error("article_cache_path(dir=%s, cid=%d) error\n", cache_dir, p_article->cid); @@ -111,18 +118,18 @@ int article_cache_generate(const char *c p_article->username, p_article->nickname, p_section->sname, p_section->stitle, p_article->title, BBS_name, str_sub_dt); snprintf(footer, sizeof(footer), - "--\n%s¡ù À´Ô´: %s http://%s [FROM: %s]\033[m\n", + "\n--\n%s¡ù À´Ô´: %s http://%s [FROM: %s]\033[m\n\n", BBS_article_footer_color[p_article->aid % BBS_article_footer_color_count], BBS_name, BBS_server, sub_ip); header_len = strnlen(header, sizeof(header)); footer_len = strnlen(footer, sizeof(footer)); - header_line_cnt = split_data_lines(header, SCREEN_COLS, cache.line_offsets, MAX_SPLIT_FILE_LINES); + header_line_cnt = split_data_lines(header, SCREEN_COLS, cache.line_offsets, MAX_SPLIT_FILE_LINES, 1); if (header_len != cache.line_offsets[header_line_cnt]) { - log_std("Header of article(aid=%d) is truncated from %ld to %ld\n", p_article->aid, header_len, cache.line_offsets[header_line_cnt]); + log_common("Header of article(aid=%d) is truncated from %ld to %ld\n", p_article->aid, header_len, cache.line_offsets[header_line_cnt]); header_len = (size_t)cache.line_offsets[header_line_cnt]; } @@ -130,11 +137,11 @@ int article_cache_generate(const char *c cache.data_len = header_len + (size_t)lml_plain(content, content_f, ARTICLE_CONTENT_MAX_LEN); cache.line_total = header_line_cnt + - split_data_lines(content_f, SCREEN_COLS, cache.line_offsets + header_line_cnt, MAX_SPLIT_FILE_LINES - header_line_cnt); + split_data_lines(content_f, SCREEN_COLS, cache.line_offsets + header_line_cnt, MAX_SPLIT_FILE_LINES - header_line_cnt, 1); if (cache.data_len - header_len != (size_t)cache.line_offsets[cache.line_total]) { - log_std("Body of article(aid=%d) is truncated from %ld to %ld\n", + log_common("Body of article(aid=%d) is truncated from %ld to %ld\n", p_article->aid, cache.data_len - header_len, cache.line_offsets[cache.line_total]); cache.data_len = header_len + (size_t)(cache.line_offsets[cache.line_total]); } @@ -144,11 +151,11 @@ int article_cache_generate(const char *c cache.line_offsets[i] += (long)header_len; } - footer_line_cnt = split_data_lines(footer, SCREEN_COLS, cache.line_offsets + cache.line_total, MAX_SPLIT_FILE_LINES - cache.line_total); + footer_line_cnt = split_data_lines(footer, SCREEN_COLS, cache.line_offsets + cache.line_total, MAX_SPLIT_FILE_LINES - cache.line_total, 1); if (footer_len != cache.line_offsets[cache.line_total + footer_line_cnt]) { - log_std("Footer of article(aid=%d) is truncated from %ld to %ld\n", + log_common("Footer of article(aid=%d) is truncated from %ld to %ld\n", p_article->aid, footer_len, cache.line_offsets[cache.line_total + footer_line_cnt]); footer_len = (size_t)(cache.line_offsets[cache.line_total + footer_line_cnt]); }