--- lbbs/src/article_cache.c	2025/05/31 12:00:00	1.5
+++ lbbs/src/article_cache.c	2025/06/01 14:04:19	1.8
@@ -31,6 +31,17 @@
 #define ARTICLE_FOOTER_MAX_LEN 4096
 #define SUB_DT_MAX_LEN 50
 
+static const char *BBS_article_footer_color[] = {
+	"\033[31m",
+	"\033[32m",
+	"\033[33m",
+	"\033[34m",
+	"\033[35m",
+	"\033[36m",
+	"\033[37m",
+};
+static const int BBS_article_footer_color_count = 7;
+
 inline static int article_cache_path(char *file_path, size_t buf_len, const char *cache_dir, const ARTICLE *p_article)
 {
 	if (file_path == NULL || cache_dir == NULL || p_article == NULL)
@@ -97,7 +108,8 @@ 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 https://%s [FROM: %s]\n",
+			 "--\n%s¡ù À´Ô´: %s http://%s [FROM: %s]\033[m\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));
@@ -108,11 +120,6 @@ int article_cache_generate(const char *c
 	header_line_cnt = split_data_lines(header, SCREEN_COLS, cache.line_offsets, MAX_SPLIT_FILE_LINES);
 	cache.line_total = header_line_cnt +
 					   split_data_lines(content, SCREEN_COLS, cache.line_offsets + header_line_cnt, MAX_SPLIT_FILE_LINES - header_line_cnt);
-	if (cache.line_total >= MAX_SPLIT_FILE_LINES)
-	{
-		log_error("split_data_lines(%s) truncated over limit lines %d >= %d\n", data_file, cache.line_total, MAX_SPLIT_FILE_LINES);
-		return -3;
-	}
 
 	for (i = header_line_cnt; i <= cache.line_total; i++)
 	{
@@ -120,11 +127,6 @@ int article_cache_generate(const char *c
 	}
 
 	footer_line_cnt = split_data_lines(footer, SCREEN_COLS, cache.line_offsets + cache.line_total, MAX_SPLIT_FILE_LINES - cache.line_total);
-	if (cache.line_total + footer_line_cnt >= MAX_SPLIT_FILE_LINES)
-	{
-		log_error("split_data_lines(%s) truncated over limit lines %d >= %d\n", data_file, cache.line_total + footer_line_cnt, MAX_SPLIT_FILE_LINES);
-		return -3;
-	}
 
 	for (i = 0; i <= footer_line_cnt; i++)
 	{
@@ -133,8 +135,9 @@ int article_cache_generate(const char *c
 
 	cache.data_len += footer_len;
 	cache.line_total += footer_line_cnt;
+	cache.mmap_len = sizeof(cache) - sizeof(long) * (MAX_SPLIT_FILE_LINES - (size_t)(cache.line_total) - 1) + cache.data_len;
 
-	if (write(fd, &cache, sizeof(cache)) == -1)
+	if (write(fd, &cache, cache.mmap_len - cache.data_len) == -1)
 	{
 		log_error("write(%s, cache) error (%d)\n", data_file, errno);
 		close(fd);
@@ -215,23 +218,16 @@ int article_cache_load(ARTICLE_CACHE *p_
 		return -2;
 	}
 
-	memcpy((void *)p_cache, p_mmap, sizeof(ARTICLE_CACHE));
-
-	if (sizeof(ARTICLE_CACHE) + p_cache->data_len != mmap_len)
+	if (((ARTICLE_CACHE *)p_mmap)->mmap_len != mmap_len)
 	{
-		log_error("Inconsistent length, cache_len(%ld) + data_len(%ld) != mmap_len(%ld)\n",
-				  sizeof(ARTICLE_CACHE), p_cache->data_len, mmap_len);
-
-		if (munmap(p_mmap, mmap_len) < 0)
-		{
-			log_error("munmap() error (%d)\n", errno);
-		}
-
+		log_error("Inconsistent mmap len of article (cid=%d), %ld != %ld\n", p_article->cid, p_cache->mmap_len, mmap_len);
 		return -3;
 	}
 
+	memcpy((void *)p_cache, p_mmap, (size_t)(((ARTICLE_CACHE *)p_mmap)->mmap_len - ((ARTICLE_CACHE *)p_mmap)->data_len));
+
 	p_cache->p_mmap = p_mmap;
-	p_cache->p_data = p_mmap + sizeof(ARTICLE_CACHE);
+	p_cache->p_data = p_mmap + (p_cache->mmap_len - p_cache->data_len);
 
 	return 0;
 }
@@ -244,7 +240,7 @@ int article_cache_unload(ARTICLE_CACHE *
 		return -1;
 	}
 
-	if (munmap(p_cache->p_mmap, sizeof(ARTICLE_CACHE) + p_cache->data_len) < 0)
+	if (munmap(p_cache->p_mmap, p_cache->mmap_len) < 0)
 	{
 		log_error("munmap() error (%d)\n", errno);
 		return -2;