/[LeafOK_CVS]/lbbs/src/section_list.c
ViewVC logotype

Diff of /lbbs/src/section_list.c

Parent Directory Parent Directory | Revision Log Revision Log | View Patch Patch

Revision 1.25 by sysadm, Mon May 26 23:38:11 2025 UTC Revision 1.49 by sysadm, Mon Nov 3 08:48:56 2025 UTC
# Line 14  Line 14 
14   *                                                                         *   *                                                                         *
15   ***************************************************************************/   ***************************************************************************/
16    
 #define _GNU_SOURCE  
   
 #include "section_list.h"  
17  #include "log.h"  #include "log.h"
18    #include "section_list.h"
19  #include "trie_dict.h"  #include "trie_dict.h"
20    #include "user_list.h"
21    #include <errno.h>
22    #include <signal.h>
23  #include <stdio.h>  #include <stdio.h>
24    #include <stdlib.h>
25  #include <string.h>  #include <string.h>
 #include <signal.h>  
26  #include <unistd.h>  #include <unistd.h>
27  #include <stdlib.h>  #include <sys/ipc.h>
 #include <errno.h>  
28  #include <sys/param.h>  #include <sys/param.h>
29  #include <sys/sem.h>  #include <sys/sem.h>
30  #include <sys/shm.h>  #include <sys/shm.h>
 #include <sys/ipc.h>  
31    
32  #ifdef _SEM_SEMUN_UNDEFINED  #ifdef _SEM_SEMUN_UNDEFINED
33  union semun  union semun
# Line 44  union semun Line 43  union semun
43  #define SECTION_TRY_LOCK_WAIT_TIME 1 // second  #define SECTION_TRY_LOCK_WAIT_TIME 1 // second
44  #define SECTION_TRY_LOCK_TIMES 10  #define SECTION_TRY_LOCK_TIMES 10
45    
46  #define ARTICLE_BLOCK_PER_SHM 400                 // sizeof(ARTICLE_BLOCK) * ARTICLE_BLOCK_PER_SHM is the size of each shm segment to allocate  #define ARTICLE_BLOCK_PER_SHM 1000               // sizeof(ARTICLE_BLOCK) * ARTICLE_BLOCK_PER_SHM is the size of each shm segment to allocate
47  #define ARTICLE_BLOCK_SHM_COUNT_LIMIT 200 // limited by length (8-bit) of proj_id in ftok(path, proj_id)  #define ARTICLE_BLOCK_SHM_COUNT_LIMIT 80 // limited by length (8-bit) of proj_id in ftok(path, proj_id)
48  #define ARTICLE_BLOCK_PER_POOL (ARTICLE_BLOCK_PER_SHM * ARTICLE_BLOCK_SHM_COUNT_LIMIT)  #define ARTICLE_BLOCK_PER_POOL (ARTICLE_BLOCK_PER_SHM * ARTICLE_BLOCK_SHM_COUNT_LIMIT)
49    
50  #define CALCULATE_PAGE_THRESHOLD 100 // Adjust to tune performance of move topic  #define CALCULATE_PAGE_THRESHOLD 100 // Adjust to tune performance of moving topic between sections
51    
52  #define SID_STR_LEN 5 // 32-bit + NULL  #define SID_STR_LEN 5 // 32-bit + NULL
53    
# Line 77  typedef struct article_block_pool_t ARTI Line 76  typedef struct article_block_pool_t ARTI
76    
77  static ARTICLE_BLOCK_POOL *p_article_block_pool = NULL;  static ARTICLE_BLOCK_POOL *p_article_block_pool = NULL;
78    
79  struct section_list_pool_t  SECTION_LIST_POOL *p_section_list_pool = NULL;
 {  
         int shmid;  
         SECTION_LIST sections[BBS_max_section];  
         int section_count;  
         int semid;  
         TRIE_NODE *p_trie_dict_section_by_name;  
         TRIE_NODE *p_trie_dict_section_by_sid;  
 };  
 typedef struct section_list_pool_t SECTION_LIST_POOL;  
   
 static SECTION_LIST_POOL *p_section_list_pool = NULL;  
80    
81  int article_block_init(const char *filename, int block_count)  int article_block_init(const char *filename, int block_count)
82  {  {
# Line 108  int article_block_init(const char *filen Line 96  int article_block_init(const char *filen
96                  return -1;                  return -1;
97          }          }
98    
99          if (block_count > ARTICLE_BLOCK_PER_POOL)          if (block_count <= 0 || block_count > ARTICLE_BLOCK_PER_POOL)
100          {          {
101                  log_error("article_block_count exceed limit %d\n", ARTICLE_BLOCK_PER_POOL);                  log_error("article_block_count exceed limit %d\n", ARTICLE_BLOCK_PER_POOL);
102                  return -2;                  return -2;
# Line 236  void article_block_cleanup(void) Line 224  void article_block_cleanup(void)
224          p_article_block_pool = NULL;          p_article_block_pool = NULL;
225  }  }
226    
227    int set_article_block_shm_readonly(void)
228    {
229            int shmid;
230            void *p_shm;
231            int i;
232    
233            if (p_article_block_pool == NULL)
234            {
235                    log_error("article_block_pool not initialized\n");
236                    return -1;
237            }
238    
239            for (i = 0; i < p_article_block_pool->shm_count; i++)
240            {
241                    shmid = (p_article_block_pool->shm_pool + i)->shmid;
242    
243                    // Remap shared memory in read-only mode
244                    p_shm = shmat(shmid, (p_article_block_pool->shm_pool + i)->p_shm, SHM_RDONLY | SHM_REMAP);
245                    if (p_shm == (void *)-1)
246                    {
247                            log_error("shmat(article_block_pool shmid = %d) error (%d)\n", shmid, errno);
248                            return -2;
249                    }
250            }
251    
252            return 0;
253    }
254    
255    int detach_article_block_shm(void)
256    {
257            int shmid;
258    
259            if (p_article_block_pool == NULL)
260            {
261                    return -1;
262            }
263    
264            for (int i = 0; i < p_article_block_pool->shm_count; i++)
265            {
266                    if ((p_article_block_pool->shm_pool + i)->p_shm != NULL && shmdt((p_article_block_pool->shm_pool + i)->p_shm) == -1)
267                    {
268                            log_error("shmdt(shmid = %d) error (%d)\n", (p_article_block_pool->shm_pool + i)->shmid, errno);
269                            return -2;
270                    }
271            }
272    
273            shmid = p_article_block_pool->shmid;
274    
275            if (shmdt(p_article_block_pool) == -1)
276            {
277                    log_error("shmdt(shmid = %d) error (%d)\n", shmid, errno);
278                    return -3;
279            }
280    
281            p_article_block_pool = NULL;
282    
283            return 0;
284    }
285    
286  inline static ARTICLE_BLOCK *pop_free_article_block(void)  inline static ARTICLE_BLOCK *pop_free_article_block(void)
287  {  {
288          ARTICLE_BLOCK *p_block = NULL;          ARTICLE_BLOCK *p_block = NULL;
# Line 296  ARTICLE *article_block_find_by_aid(int32 Line 343  ARTICLE *article_block_find_by_aid(int32
343          }          }
344    
345          left = 0;          left = 0;
346          right = p_article_block_pool->block_count;          right = p_article_block_pool->block_count - 1;
347    
348          // aid in the range [ head aid of blocks[left], tail aid of blocks[right - 1] ]          // aid in the range [ head aid of blocks[left], tail aid of blocks[right] ]
349          while (left < right - 1)          while (left < right)
350          {          {
351                  // get block offset no less than mid value of left and right block offsets                  // get block offset no less than mid value of left and right block offsets
352                  mid = (left + right) / 2 + (right - left) % 2;                  mid = (left + right) / 2 + (left + right) % 2;
   
                 if (mid >= p_article_block_pool->block_count)  
                 {  
                         log_error("block(mid = %d) is out of boundary\n", mid);  
                         return NULL;  
                 }  
353    
354                  if (aid < p_article_block_pool->p_block[mid]->articles[0].aid)                  if (aid < p_article_block_pool->p_block[mid]->articles[0].aid)
355                  {                  {
356                          right = mid;                          right = mid - 1;
357                  }                  }
358                  else                  else // if (aid >= p_article_block_pool->p_block[mid]->articles[0].aid)
359                  {                  {
360                          left = mid;                          left = mid;
361                  }                  }
# Line 340  ARTICLE *article_block_find_by_aid(int32 Line 381  ARTICLE *article_block_find_by_aid(int32
381                  }                  }
382          }          }
383    
384          return (p_block->articles + left);          if (aid != p_block->articles[left].aid) // not found
385            {
386                    return NULL;
387            }
388    
389            return (p_block->articles + left); // found
390  }  }
391    
392  ARTICLE *article_block_find_by_index(int index)  ARTICLE *article_block_find_by_index(int index)
# Line 452  extern int section_list_init(const char Line 498  extern int section_list_init(const char
498          return 0;          return 0;
499  }  }
500    
501    void section_list_cleanup(void)
502    {
503            int shmid;
504    
505            if (p_section_list_pool == NULL)
506            {
507                    return;
508            }
509    
510            if (p_section_list_pool->p_trie_dict_section_by_name != NULL)
511            {
512                    trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_name);
513                    p_section_list_pool->p_trie_dict_section_by_name = NULL;
514            }
515    
516            if (p_section_list_pool->p_trie_dict_section_by_sid != NULL)
517            {
518                    trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_sid);
519                    p_section_list_pool->p_trie_dict_section_by_sid = NULL;
520            }
521    
522            shmid = p_section_list_pool->shmid;
523    
524            if (semctl(p_section_list_pool->semid, 0, IPC_RMID) == -1)
525            {
526                    log_error("semctl(semid = %d, IPC_RMID) error (%d)\n", p_section_list_pool->semid, errno);
527            }
528    
529            if (shmdt(p_section_list_pool) == -1)
530            {
531                    log_error("shmdt(shmid = %d) error (%d)\n", shmid, errno);
532            }
533    
534            if (shmctl(shmid, IPC_RMID, NULL) == -1)
535            {
536                    log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", shmid, errno);
537            }
538    
539            p_section_list_pool = NULL;
540    }
541    
542    int set_section_list_shm_readonly(void)
543    {
544            int shmid;
545            void *p_shm;
546    
547            if (p_section_list_pool == NULL)
548            {
549                    log_error("p_section_list_pool not initialized\n");
550                    return -1;
551            }
552    
553            shmid = p_section_list_pool->shmid;
554    
555            // Remap shared memory in read-only mode
556            p_shm = shmat(shmid, p_section_list_pool, SHM_RDONLY | SHM_REMAP);
557            if (p_shm == (void *)-1)
558            {
559                    log_error("shmat(section_list_pool shmid = %d) error (%d)\n", shmid, errno);
560                    return -3;
561            }
562    
563            p_section_list_pool = p_shm;
564    
565            return 0;
566    }
567    
568    int detach_section_list_shm(void)
569    {
570            if (p_section_list_pool != NULL && shmdt(p_section_list_pool) == -1)
571            {
572                    log_error("shmdt(section_list_pool) error (%d)\n", errno);
573                    return -1;
574            }
575    
576            p_section_list_pool = NULL;
577    
578            return 0;
579    }
580    
581  inline static void sid_to_str(int32_t sid, char *p_sid_str)  inline static void sid_to_str(int32_t sid, char *p_sid_str)
582  {  {
583          uint32_t u_sid;          uint32_t u_sid;
# Line 466  inline static void sid_to_str(int32_t si Line 592  inline static void sid_to_str(int32_t si
592          p_sid_str[i] = '\0';          p_sid_str[i] = '\0';
593  }  }
594    
595  SECTION_LIST *section_list_create(int32_t sid, const char *sname, const char *stitle, const char *master_name)  SECTION_LIST *section_list_create(int32_t sid, const char *sname, const char *stitle, const char *master_list)
596  {  {
597          SECTION_LIST *p_section;          SECTION_LIST *p_section;
598          char sid_str[SID_STR_LEN];          char sid_str[SID_STR_LEN];
# Line 488  SECTION_LIST *section_list_create(int32_ Line 614  SECTION_LIST *section_list_create(int32_
614          p_section = p_section_list_pool->sections + p_section_list_pool->section_count;          p_section = p_section_list_pool->sections + p_section_list_pool->section_count;
615    
616          p_section->sid = sid;          p_section->sid = sid;
617            p_section->ex_menu_tm = 0;
618    
619          strncpy(p_section->sname, sname, sizeof(p_section->sname - 1));          strncpy(p_section->sname, sname, sizeof(p_section->sname) - 1);
620          p_section->sname[sizeof(p_section->sname - 1)] = '\0';          p_section->sname[sizeof(p_section->sname) - 1] = '\0';
621    
622          strncpy(p_section->stitle, stitle, sizeof(p_section->stitle - 1));          strncpy(p_section->stitle, stitle, sizeof(p_section->stitle) - 1);
623          p_section->stitle[sizeof(p_section->stitle - 1)] = '\0';          p_section->stitle[sizeof(p_section->stitle) - 1] = '\0';
624    
625          strncpy(p_section->master_name, master_name, sizeof(p_section->master_name - 1));          strncpy(p_section->master_list, master_list, sizeof(p_section->master_list) - 1);
626          p_section->master_name[sizeof(p_section->master_name - 1)] = '\0';          p_section->master_list[sizeof(p_section->master_list) - 1] = '\0';
627    
628          if (trie_dict_set(p_section_list_pool->p_trie_dict_section_by_name, sname, p_section_list_pool->section_count) != 1)          if (trie_dict_set(p_section_list_pool->p_trie_dict_section_by_name, sname, p_section_list_pool->section_count) != 1)
629          {          {
# Line 517  SECTION_LIST *section_list_create(int32_ Line 644  SECTION_LIST *section_list_create(int32_
644          return p_section;          return p_section;
645  }  }
646    
647  void section_list_reset_articles(SECTION_LIST *p_section)  int section_list_update(SECTION_LIST *p_section, const char *sname, const char *stitle, const char *master_list)
 {  
         p_section->article_count = 0;  
         p_section->topic_count = 0;  
         p_section->visible_article_count = 0;  
         p_section->visible_topic_count = 0;  
         p_section->p_article_head = NULL;  
         p_section->p_article_tail = NULL;  
   
         p_section->page_count = 0;  
         p_section->last_page_visible_article_count = 0;  
 }  
   
 void section_list_cleanup(void)  
648  {  {
649          int shmid;          int64_t index;
650    
651          if (p_section_list_pool == NULL)          if (p_section == NULL || sname == NULL || stitle == NULL || master_list == NULL)
652          {          {
653                  return;                  log_error("NULL pointer error\n");
654                    return -1;
655          }          }
656    
657          if (p_section_list_pool->p_trie_dict_section_by_name != NULL)          index = get_section_index(p_section);
         {  
                 trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_name);  
                 p_section_list_pool->p_trie_dict_section_by_name = NULL;  
         }  
658    
659          if (p_section_list_pool->p_trie_dict_section_by_sid != NULL)          strncpy(p_section->sname, sname, sizeof(p_section->sname) - 1);
660          {          p_section->sname[sizeof(p_section->sname) - 1] = '\0';
                 trie_dict_destroy(p_section_list_pool->p_trie_dict_section_by_sid);  
                 p_section_list_pool->p_trie_dict_section_by_sid = NULL;  
         }  
661    
662          if (p_section_list_pool != NULL)          strncpy(p_section->stitle, stitle, sizeof(p_section->stitle) - 1);
663            p_section->stitle[sizeof(p_section->stitle) - 1] = '\0';
664    
665            strncpy(p_section->master_list, master_list, sizeof(p_section->master_list) - 1);
666            p_section->master_list[sizeof(p_section->master_list) - 1] = '\0';
667    
668            if (trie_dict_set(p_section_list_pool->p_trie_dict_section_by_name, sname, index) < 0)
669          {          {
670                  if (semctl(p_section_list_pool->semid, 0, IPC_RMID) == -1)                  log_error("trie_dict_set(section, %s, %d) error\n", sname, index);
671                  {                  return -2;
672                          log_error("semctl(semid = %d, IPC_RMID) error (%d)\n", p_section_list_pool->semid, errno);          }
                 }  
673    
674                  shmid = p_section_list_pool->shmid;          return 0;
675    }
676    
677                  if (shmdt(p_section_list_pool) == -1)  void section_list_reset_articles(SECTION_LIST *p_section)
678                  {  {
679                          log_error("shmdt(shmid = %d) error (%d)\n", shmid, errno);          p_section->article_count = 0;
680                  }          p_section->topic_count = 0;
681            p_section->visible_article_count = 0;
682            p_section->visible_topic_count = 0;
683            p_section->p_article_head = NULL;
684            p_section->p_article_tail = NULL;
685    
686                  if (shmctl(shmid, IPC_RMID, NULL) == -1)          p_section->page_count = 0;
687                  {          p_section->last_page_visible_article_count = 0;
                         log_error("shmctl(shmid = %d, IPC_RMID) error (%d)\n", shmid, errno);  
                 }  
688    
689                  p_section_list_pool = NULL;          p_section->ontop_article_count = 0;
         }  
690  }  }
691    
692  SECTION_LIST *section_list_find_by_name(const char *sname)  SECTION_LIST *section_list_find_by_name(const char *sname)
# Line 637  int section_list_append_article(SECTION_ Line 752  int section_list_append_article(SECTION_
752    
753          if (p_section == NULL || p_article_src == NULL)          if (p_section == NULL || p_article_src == NULL)
754          {          {
755                  log_error("section_list_append_article() NULL pointer error\n");                  log_error("NULL pointer error\n");
756                  return -1;                  return -1;
757          }          }
758    
# Line 762  int section_list_append_article(SECTION_ Line 877  int section_list_append_article(SECTION_
877                  p_section->last_page_visible_article_count++;                  p_section->last_page_visible_article_count++;
878          }          }
879    
880            if (p_article->ontop && section_list_update_article_ontop(p_section, p_article) < 0)
881            {
882                    log_error("section_list_update_article_ontop(sid=%d, aid=%d) error\n",
883                                      p_section->sid, p_article->aid);
884                    return -5;
885            }
886    
887          return 0;          return 0;
888  }  }
889    
# Line 773  int section_list_set_article_visible(SEC Line 895  int section_list_set_article_visible(SEC
895    
896          if (p_section == NULL)          if (p_section == NULL)
897          {          {
898                  log_error("section_list_set_article_visible() NULL pointer error\n");                  log_error("NULL pointer error\n");
899                  return -2;                  return -1;
900          }          }
901    
902          p_article = article_block_find_by_aid(aid);          p_article = article_block_find_by_aid(aid);
# Line 785  int section_list_set_article_visible(SEC Line 907  int section_list_set_article_visible(SEC
907    
908          if (p_section->sid != p_article->sid)          if (p_section->sid != p_article->sid)
909          {          {
910                  log_error("section_list_set_article_visible() error: section sid %d != article sid %d\n", p_section->sid, p_article->sid);                  log_error("Inconsistent section sid %d != article sid %d\n", p_section->sid, p_article->sid);
911                  return -2;                  return -2;
912          }          }
913    
# Line 798  int section_list_set_article_visible(SEC Line 920  int section_list_set_article_visible(SEC
920          {          {
921                  p_section->visible_article_count--;                  p_section->visible_article_count--;
922    
923                    if (user_article_cnt_inc(p_article->uid, -1) < 0)
924                    {
925                            log_error("user_article_cnt_inc(uid=%d, -1) error\n", p_article->uid);
926                    }
927    
928                  if (p_article->tid == 0)                  if (p_article->tid == 0)
929                  {                  {
930                          p_section->visible_topic_count--;                          p_section->visible_topic_count--;
# Line 816  int section_list_set_article_visible(SEC Line 943  int section_list_set_article_visible(SEC
943                                          p_reply->visible = 0;                                          p_reply->visible = 0;
944                                          p_section->visible_article_count--;                                          p_section->visible_article_count--;
945                                          affected_count++;                                          affected_count++;
946    
947                                            if (user_article_cnt_inc(p_reply->uid, -1) < 0)
948                                            {
949                                                    log_error("user_article_cnt_inc(uid=%d, -1) error\n", p_reply->uid);
950                                            }
951                                  }                                  }
952                          }                          }
953                  }                  }
# Line 828  int section_list_set_article_visible(SEC Line 960  int section_list_set_article_visible(SEC
960                  {                  {
961                          p_section->visible_topic_count++;                          p_section->visible_topic_count++;
962                  }                  }
963    
964                    if (user_article_cnt_inc(p_article->uid, 1) < 0)
965                    {
966                            log_error("user_article_cnt_inc(uid=%d, 1) error\n", p_article->uid);
967                    }
968          }          }
969    
970          p_article->visible = visible;          p_article->visible = visible;
# Line 836  int section_list_set_article_visible(SEC Line 973  int section_list_set_article_visible(SEC
973          return affected_count;          return affected_count;
974  }  }
975    
976    int section_list_update_article_ontop(SECTION_LIST *p_section, ARTICLE *p_article)
977    {
978            int i;
979    
980            if (p_section == NULL || p_article == NULL)
981            {
982                    log_error("NULL pointer error\n");
983                    return -1;
984            }
985    
986            if (p_section->sid != p_article->sid)
987            {
988                    log_error("Inconsistent section sid %d != article sid %d\n", p_section->sid, p_article->sid);
989                    return -2;
990            }
991    
992            if (p_article->ontop)
993            {
994                    for (i = 0; i < p_section->ontop_article_count; i++)
995                    {
996                            if (p_section->p_ontop_articles[i]->aid == p_article->aid)
997                            {
998                                    log_error("Inconsistent state found: article %d already ontop in section %d\n", p_article->aid, p_section->sid);
999                                    return 0;
1000                            }
1001                            else if (p_section->p_ontop_articles[i]->aid > p_article->aid)
1002                            {
1003                                    break;
1004                            }
1005                    }
1006    
1007                    // Remove the oldest one if the array of ontop articles is full
1008                    if (p_section->ontop_article_count >= BBS_ontop_article_limit_per_section)
1009                    {
1010                            if (i == 0) // p_article is the oldest one
1011                            {
1012                                    return 0;
1013                            }
1014                            memmove((void *)(p_section->p_ontop_articles),
1015                                            (void *)(p_section->p_ontop_articles + 1),
1016                                            sizeof(ARTICLE *) * (size_t)(i - 1));
1017                            p_section->ontop_article_count--;
1018                            i--;
1019                    }
1020                    else
1021                    {
1022                            memmove((void *)(p_section->p_ontop_articles + i + 1),
1023                                            (void *)(p_section->p_ontop_articles + i),
1024                                            sizeof(ARTICLE *) * (size_t)(p_section->ontop_article_count - i));
1025                    }
1026    
1027                    p_section->p_ontop_articles[i] = p_article;
1028                    p_section->ontop_article_count++;
1029    
1030                    // TODO: debug
1031            }
1032            else // ontop == 0
1033            {
1034                    for (i = 0; i < p_section->ontop_article_count; i++)
1035                    {
1036                            if (p_section->p_ontop_articles[i]->aid == p_article->aid)
1037                            {
1038                                    break;
1039                            }
1040                    }
1041                    if (i == p_section->ontop_article_count) // not found
1042                    {
1043                            log_error("Inconsistent state found: article %d not ontop in section %d\n", p_article->aid, p_section->sid);
1044                            return 0;
1045                    }
1046    
1047                    memmove((void *)(p_section->p_ontop_articles + i),
1048                                    (void *)(p_section->p_ontop_articles + i + 1),
1049                                    sizeof(ARTICLE *) * (size_t)(p_section->ontop_article_count - i - 1));
1050                    p_section->ontop_article_count--;
1051            }
1052    
1053            return 0;
1054    }
1055    
1056    int section_list_page_count_with_ontop(SECTION_LIST *p_section)
1057    {
1058            int page_count;
1059    
1060            if (p_section == NULL)
1061            {
1062                    log_error("NULL pointer error\n");
1063                    return -1;
1064            }
1065    
1066            page_count = p_section->page_count - (p_section->last_page_visible_article_count > 0 ? 1 : 0) +
1067                                     (p_section->last_page_visible_article_count + p_section->ontop_article_count) / BBS_article_limit_per_page +
1068                                     ((p_section->last_page_visible_article_count + p_section->ontop_article_count) % BBS_article_limit_per_page == 0 ? 0 : 1);
1069    
1070            return page_count;
1071    }
1072    
1073    int section_list_page_article_count_with_ontop(SECTION_LIST *p_section, int32_t page_id)
1074    {
1075            if (p_section == NULL)
1076            {
1077                    log_error("NULL pointer error\n");
1078                    return -1;
1079            }
1080    
1081            if (page_id < p_section->page_count - 1)
1082            {
1083                    return BBS_article_limit_per_page;
1084            }
1085            else // if (page_id >= p_section->page_count - 1)
1086            {
1087                    return MIN(MAX(0,
1088                                               (p_section->last_page_visible_article_count + p_section->ontop_article_count -
1089                                                    BBS_article_limit_per_page * (page_id - p_section->page_count + 1))),
1090                                       BBS_article_limit_per_page);
1091            }
1092    }
1093    
1094  ARTICLE *section_list_find_article_with_offset(SECTION_LIST *p_section, int32_t aid, int32_t *p_page, int32_t *p_offset, ARTICLE **pp_next)  ARTICLE *section_list_find_article_with_offset(SECTION_LIST *p_section, int32_t aid, int32_t *p_page, int32_t *p_offset, ARTICLE **pp_next)
1095  {  {
1096          ARTICLE *p_article;          ARTICLE *p_article;
# Line 849  ARTICLE *section_list_find_article_with_ Line 1104  ARTICLE *section_list_find_article_with_
1104    
1105          if (p_section == NULL)          if (p_section == NULL)
1106          {          {
1107                  log_error("section_list_find_article_with_offset() NULL pointer error\n");                  log_error("NULL pointer error\n");
1108                  return NULL;                  return NULL;
1109          }          }
1110    
# Line 861  ARTICLE *section_list_find_article_with_ Line 1116  ARTICLE *section_list_find_article_with_
1116          }          }
1117    
1118          left = 0;          left = 0;
1119          right = p_section->page_count;          right = p_section->page_count - 1;
1120    
1121          // aid in the range [ head aid of pages[left], tail aid of pages[right - 1] ]          // aid in the range [ head aid of pages[left], tail aid of pages[right] ]
1122          while (left < right - 1)          while (left < right)
1123          {          {
1124                  // get page id no less than mid value of left page id and right page id                  // get page id no less than mid value of left page id and right page id
1125                  mid = (left + right) / 2 + (right - left) % 2;                  mid = (left + right) / 2 + (left + right) % 2;
   
                 if (mid >= p_section->page_count)  
                 {  
                         log_error("page id (mid = %d) is out of boundary\n", mid);  
                         return NULL;  
                 }  
1126    
1127                  if (aid < p_section->p_page_first_article[mid]->aid)                  if (aid < p_section->p_page_first_article[mid]->aid)
1128                  {                  {
1129                          right = mid;                          right = mid - 1;
1130                  }                  }
1131                  else                  else // if (aid < p_section->p_page_first_article[mid]->aid)
1132                  {                  {
1133                          left = mid;                          left = mid;
1134                  }                  }
# Line 941  int section_list_calculate_page(SECTION_ Line 1190  int section_list_calculate_page(SECTION_
1190    
1191          if (p_section == NULL)          if (p_section == NULL)
1192          {          {
1193                  log_error("section_list_calculate_page() NULL pointer error\n");                  log_error("NULL pointer error\n");
1194                  return -1;                  return -1;
1195          }          }
1196    
# Line 1043  int32_t article_block_last_aid(void) Line 1292  int32_t article_block_last_aid(void)
1292          return last_aid;          return last_aid;
1293  }  }
1294    
1295    int article_block_article_count(void)
1296    {
1297            int ret;
1298    
1299            if (p_article_block_pool == NULL || p_article_block_pool->block_count <= 0)
1300            {
1301                    return -1;
1302            }
1303    
1304            ret = (p_article_block_pool->block_count - 1) * ARTICLE_PER_BLOCK +
1305                      p_article_block_pool->p_block[p_article_block_pool->block_count - 1]->article_count;
1306    
1307            return ret;
1308    }
1309    
1310  int article_count_of_topic(int32_t aid)  int article_count_of_topic(int32_t aid)
1311  {  {
1312          ARTICLE *p_article;          ARTICLE *p_article;
# Line 1083  int section_list_move_topic(SECTION_LIST Line 1347  int section_list_move_topic(SECTION_LIST
1347          int32_t first_inserted_aid_dest;          int32_t first_inserted_aid_dest;
1348          int move_counter;          int move_counter;
1349    
1350          if (p_section_dest == NULL)          if (p_section_src == NULL || p_section_dest == NULL)
1351          {          {
1352                  log_error("section_list_move_topic() NULL pointer error\n");                  log_error("NULL pointer error\n");
1353                    return -1;
1354            }
1355    
1356            if (p_section_src->sid == p_section_dest->sid)
1357            {
1358                    log_error("section_list_move_topic() src and dest section are the same\n");
1359                  return -1;                  return -1;
1360          }          }
1361    
1362          if ((p_article = article_block_find_by_aid(aid)) == NULL)          if ((p_article = article_block_find_by_aid(aid)) == NULL)
1363          {          {
1364                  log_error("section_list_move_topic() error: article %d not found in block\n", aid);                  log_error("article_block_find_by_aid(aid = %d) error: article not found\n", aid);
1365                  return -2;                  return -2;
1366          }          }
1367    
# Line 1132  int section_list_move_topic(SECTION_LIST Line 1402  int section_list_move_topic(SECTION_LIST
1402          {          {
1403                  if (p_section_src->sid != p_article->sid)                  if (p_section_src->sid != p_article->sid)
1404                  {                  {
1405                          log_error("section_list_move_topic() error: src section sid %d != article %d sid %d\n",                          log_error("section_list_move_topic() warning: src section sid %d != article %d sid %d\n",
1406                                            p_section_src->sid, p_article->aid, p_article->sid);                                            p_section_src->sid, p_article->aid, p_article->sid);
1407                          return -2;                          p_article = p_article->p_topic_next;
1408                            continue;
1409                  }                  }
1410    
1411                  // Remove from bi-directional article list of src section                  // Remove from bi-directional article list of src section
# Line 1160  int section_list_move_topic(SECTION_LIST Line 1431  int section_list_move_topic(SECTION_LIST
1431    
1432                  if (section_list_find_article_with_offset(p_section_dest, p_article->aid, &page, &offset, &p_next) != NULL)                  if (section_list_find_article_with_offset(p_section_dest, p_article->aid, &page, &offset, &p_next) != NULL)
1433                  {                  {
1434                          log_error("section_list_move_topic() error: article %d already in section %d\n", p_article->aid, p_section_dest->sid);                          log_error("section_list_move_topic() warning: article %d already in section %d\n", p_article->aid, p_section_dest->sid);
1435                          return -4;                          p_article = p_article->p_topic_next;
1436                            continue;
1437                  }                  }
1438    
1439                  // Insert into bi-directional article list of dest section                  // Insert into bi-directional article list of dest section
# Line 1239  int section_list_move_topic(SECTION_LIST Line 1511  int section_list_move_topic(SECTION_LIST
1511    
1512          if (p_section_dest->article_count - dest_article_count_old != move_article_count)          if (p_section_dest->article_count - dest_article_count_old != move_article_count)
1513          {          {
1514                  log_error("section_list_move_topic() error: count of moved articles %d != %d\n",                  log_error("section_list_move_topic() warning: count of moved articles %d != %d\n",
1515                                    p_section_dest->article_count - dest_article_count_old, move_article_count);                                    p_section_dest->article_count - dest_article_count_old, move_article_count);
1516          }          }
1517    
# Line 1290  int get_section_index(SECTION_LIST *p_se Line 1562  int get_section_index(SECTION_LIST *p_se
1562          return index;          return index;
1563  }  }
1564    
1565    int get_section_info(SECTION_LIST *p_section, char *sname, char *stitle, char *master_list)
1566    {
1567            if (p_section == NULL)
1568            {
1569                    log_error("NULL pointer error\n");
1570                    return -1;
1571            }
1572    
1573            if (section_list_rd_lock(p_section) < 0)
1574            {
1575                    log_error("section_list_rd_lock(sid=%d) error\n", p_section->sid);
1576                    return -2;
1577            }
1578    
1579            if (sname != NULL)
1580            {
1581                    memcpy(sname, p_section->sname, sizeof(p_section->sname));
1582            }
1583            if (stitle != NULL)
1584            {
1585                    memcpy(stitle, p_section->stitle, sizeof(p_section->stitle));
1586            }
1587            if (master_list != NULL)
1588            {
1589                    memcpy(master_list, p_section->master_list, sizeof(p_section->master_list));
1590            }
1591    
1592            // release lock of section
1593            if (section_list_rd_unlock(p_section) < 0)
1594            {
1595                    log_error("section_list_rd_unlock(sid=%d) error\n", p_section->sid);
1596                    return -2;
1597            }
1598    
1599            return 0;
1600    }
1601    
1602  int section_list_try_rd_lock(SECTION_LIST *p_section, int wait_sec)  int section_list_try_rd_lock(SECTION_LIST *p_section, int wait_sec)
1603  {  {
1604          int index;          int index;
# Line 1449  int section_list_rd_lock(SECTION_LIST *p Line 1758  int section_list_rd_lock(SECTION_LIST *p
1758                          timer++;                          timer++;
1759                          if (timer % SECTION_TRY_LOCK_TIMES == 0)                          if (timer % SECTION_TRY_LOCK_TIMES == 0)
1760                          {                          {
1761                                  log_error("section_list_rd_lock() tried %d times on section %d\n", sid, timer);                                  log_error("section_list_try_rd_lock() tried %d times on section %d\n", timer, sid);
1762                          }                          }
1763                  }                  }
1764                  else // failed                  else // failed
1765                  {                  {
1766                          log_error("section_list_rd_lock() failed on section %d\n", sid);                          log_error("section_list_try_rd_lock() failed on section %d\n", sid);
1767                          break;                          break;
1768                  }                  }
1769          }          }
# Line 1480  int section_list_rw_lock(SECTION_LIST *p Line 1789  int section_list_rw_lock(SECTION_LIST *p
1789                          timer++;                          timer++;
1790                          if (timer % SECTION_TRY_LOCK_TIMES == 0)                          if (timer % SECTION_TRY_LOCK_TIMES == 0)
1791                          {                          {
1792                                  log_error("acquire_section_rw_lock() tried %d times on section %d\n", sid, timer);                                  log_error("section_list_try_rw_lock() tried %d times on section %d\n", timer, sid);
1793                          }                          }
1794                  }                  }
1795                  else // failed                  else // failed
1796                  {                  {
1797                          log_error("acquire_section_rw_lock() failed on section %d\n", sid);                          log_error("section_list_try_rw_lock() failed on section %d\n", sid);
1798                          break;                          break;
1799                  }                  }
1800          }          }

Legend:
Removed lines/characters  
Changed lines/characters
  Added lines/characters
webmaster@leafok.com
 ViewVC Help
Powered by ViewVC 1.3.0-beta1