/[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.47 by sysadm, Fri Oct 24 02:08:19 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 MAX(0, (p_section->last_page_visible_article_count + p_section->ontop_article_count -
1088                                               BBS_article_limit_per_page * (page_id - p_section->page_count + 1)));
1089            }
1090    }
1091    
1092  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)
1093  {  {
1094          ARTICLE *p_article;          ARTICLE *p_article;
# Line 849  ARTICLE *section_list_find_article_with_ Line 1102  ARTICLE *section_list_find_article_with_
1102    
1103          if (p_section == NULL)          if (p_section == NULL)
1104          {          {
1105                  log_error("section_list_find_article_with_offset() NULL pointer error\n");                  log_error("NULL pointer error\n");
1106                  return NULL;                  return NULL;
1107          }          }
1108    
# Line 861  ARTICLE *section_list_find_article_with_ Line 1114  ARTICLE *section_list_find_article_with_
1114          }          }
1115    
1116          left = 0;          left = 0;
1117          right = p_section->page_count;          right = p_section->page_count - 1;
1118    
1119          // 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] ]
1120          while (left < right - 1)          while (left < right)
1121          {          {
1122                  // 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
1123                  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;  
                 }  
1124    
1125                  if (aid < p_section->p_page_first_article[mid]->aid)                  if (aid < p_section->p_page_first_article[mid]->aid)
1126                  {                  {
1127                          right = mid;                          right = mid - 1;
1128                  }                  }
1129                  else                  else // if (aid < p_section->p_page_first_article[mid]->aid)
1130                  {                  {
1131                          left = mid;                          left = mid;
1132                  }                  }
# Line 941  int section_list_calculate_page(SECTION_ Line 1188  int section_list_calculate_page(SECTION_
1188    
1189          if (p_section == NULL)          if (p_section == NULL)
1190          {          {
1191                  log_error("section_list_calculate_page() NULL pointer error\n");                  log_error("NULL pointer error\n");
1192                  return -1;                  return -1;
1193          }          }
1194    
# Line 1043  int32_t article_block_last_aid(void) Line 1290  int32_t article_block_last_aid(void)
1290          return last_aid;          return last_aid;
1291  }  }
1292    
1293    int article_block_article_count(void)
1294    {
1295            int ret;
1296    
1297            if (p_article_block_pool == NULL || p_article_block_pool->block_count <= 0)
1298            {
1299                    return -1;
1300            }
1301    
1302            ret = (p_article_block_pool->block_count - 1) * ARTICLE_PER_BLOCK +
1303                      p_article_block_pool->p_block[p_article_block_pool->block_count - 1]->article_count;
1304    
1305            return ret;
1306    }
1307    
1308  int article_count_of_topic(int32_t aid)  int article_count_of_topic(int32_t aid)
1309  {  {
1310          ARTICLE *p_article;          ARTICLE *p_article;
# Line 1083  int section_list_move_topic(SECTION_LIST Line 1345  int section_list_move_topic(SECTION_LIST
1345          int32_t first_inserted_aid_dest;          int32_t first_inserted_aid_dest;
1346          int move_counter;          int move_counter;
1347    
1348          if (p_section_dest == NULL)          if (p_section_src == NULL || p_section_dest == NULL)
1349          {          {
1350                  log_error("section_list_move_topic() NULL pointer error\n");                  log_error("NULL pointer error\n");
1351                    return -1;
1352            }
1353    
1354            if (p_section_src->sid == p_section_dest->sid)
1355            {
1356                    log_error("section_list_move_topic() src and dest section are the same\n");
1357                  return -1;                  return -1;
1358          }          }
1359    
1360          if ((p_article = article_block_find_by_aid(aid)) == NULL)          if ((p_article = article_block_find_by_aid(aid)) == NULL)
1361          {          {
1362                  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);
1363                  return -2;                  return -2;
1364          }          }
1365    
# Line 1132  int section_list_move_topic(SECTION_LIST Line 1400  int section_list_move_topic(SECTION_LIST
1400          {          {
1401                  if (p_section_src->sid != p_article->sid)                  if (p_section_src->sid != p_article->sid)
1402                  {                  {
1403                          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",
1404                                            p_section_src->sid, p_article->aid, p_article->sid);                                            p_section_src->sid, p_article->aid, p_article->sid);
1405                          return -2;                          p_article = p_article->p_topic_next;
1406                            continue;
1407                  }                  }
1408    
1409                  // 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 1429  int section_list_move_topic(SECTION_LIST
1429    
1430                  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)
1431                  {                  {
1432                          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);
1433                          return -4;                          p_article = p_article->p_topic_next;
1434                            continue;
1435                  }                  }
1436    
1437                  // 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 1509  int section_list_move_topic(SECTION_LIST
1509    
1510          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)
1511          {          {
1512                  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",
1513                                    p_section_dest->article_count - dest_article_count_old, move_article_count);                                    p_section_dest->article_count - dest_article_count_old, move_article_count);
1514          }          }
1515    
# Line 1449  int section_list_rd_lock(SECTION_LIST *p Line 1719  int section_list_rd_lock(SECTION_LIST *p
1719                          timer++;                          timer++;
1720                          if (timer % SECTION_TRY_LOCK_TIMES == 0)                          if (timer % SECTION_TRY_LOCK_TIMES == 0)
1721                          {                          {
1722                                  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);
1723                          }                          }
1724                  }                  }
1725                  else // failed                  else // failed
1726                  {                  {
1727                          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);
1728                          break;                          break;
1729                  }                  }
1730          }          }
# Line 1480  int section_list_rw_lock(SECTION_LIST *p Line 1750  int section_list_rw_lock(SECTION_LIST *p
1750                          timer++;                          timer++;
1751                          if (timer % SECTION_TRY_LOCK_TIMES == 0)                          if (timer % SECTION_TRY_LOCK_TIMES == 0)
1752                          {                          {
1753                                  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);
1754                          }                          }
1755                  }                  }
1756                  else // failed                  else // failed
1757                  {                  {
1758                          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);
1759                          break;                          break;
1760                  }                  }
1761          }          }

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