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

Diff of /lbbs/src/section_list_loader.c

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

Revision 1.21 by sysadm, Sun Jun 15 02:24:45 2025 UTC Revision 1.34 by sysadm, Tue Jun 24 10:01:24 2025 UTC
# Line 14  Line 14 
14   *                                                                         *   *                                                                         *
15   ***************************************************************************/   ***************************************************************************/
16    
 #include "section_list_loader.h"  
17  #include "article_cache.h"  #include "article_cache.h"
18  #include "bbs.h"  #include "bbs.h"
 #include "log.h"  
19  #include "database.h"  #include "database.h"
20    #include "log.h"
21  #include "menu.h"  #include "menu.h"
22  #include <stdio.h>  #include "section_list_loader.h"
 #include <string.h>  
23  #include <errno.h>  #include <errno.h>
24  #include <signal.h>  #include <signal.h>
25    #include <stdio.h>
26  #include <stdlib.h>  #include <stdlib.h>
 #include <strings.h>  
 #include <unistd.h>  
   
 #define _POSIX_C_SOURCE 200809L  
27  #include <string.h>  #include <string.h>
28    #include <unistd.h>
29    
30  int section_list_loader_pid;  int section_list_loader_pid;
31  int last_article_op_log_mid;  int last_article_op_log_mid;
32    
33  int load_section_config_from_db(void)  int load_section_config_from_db(int reload)
34  {  {
35          MYSQL *db;          MYSQL *db = NULL;
36          MYSQL_RES *rs, *rs2;          MYSQL_RES *rs = NULL, *rs2 = NULL;
37          MYSQL_ROW row, row2;          MYSQL_ROW row, row2;
38          char sql[SQL_BUFFER_LEN];          char sql[SQL_BUFFER_LEN];
39          int32_t sid;          int32_t sid;
40          char master_list[(BBS_username_max_len + 1) * 3 + 1];          char master_list[(BBS_username_max_len + 1) * 3 + 1];
41          SECTION_LIST *p_section;          SECTION_LIST *p_section;
42          int ret;          char ex_menu_conf[FILE_PATH_LEN];
43            MENU_SET ex_menu_set_new;
44            int ret = 0;
45    
46          db = db_open();          db = db_open();
47          if (db == NULL)          if (db == NULL)
48          {          {
49                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s\n", mysql_error(db));
50                  return -2;                  ret = -1;
51                    goto cleanup;
52          }          }
53    
54          snprintf(sql, sizeof(sql),          snprintf(sql, sizeof(sql),
55                           "SELECT section_config.SID, sname, section_config.title, section_config.CID, "                           "SELECT section_config.SID, sname, section_config.title, section_config.CID, "
56                           "read_user_level, write_user_level, section_config.enable * section_class.enable AS enable "                           "read_user_level, write_user_level, section_config.enable * section_class.enable AS enable, "
57                             "UNIX_TIMESTAMP(ex_menu_tm) AS ex_menu_tm "
58                           "FROM section_config INNER JOIN section_class ON section_config.CID = section_class.CID "                           "FROM section_config INNER JOIN section_class ON section_config.CID = section_class.CID "
59                           "ORDER BY section_config.SID");                           "ORDER BY section_config.SID");
60    
61          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
62          {          {
63                  log_error("Query section_list error: %s\n", mysql_error(db));                  log_error("Query section_list error: %s\n", mysql_error(db));
64                  return -3;                  ret = -2;
65                    goto cleanup;
66          }          }
67          if ((rs = mysql_store_result(db)) == NULL)          if ((rs = mysql_store_result(db)) == NULL)
68          {          {
69                  log_error("Get section_list data failed\n");                  log_error("Get section_list data failed\n");
70                  return -3;                  ret = -2;
71                    goto cleanup;
72          }          }
73    
74          ret = 0;          ret = 0;
# Line 102  int load_section_config_from_db(void) Line 104  int load_section_config_from_db(void)
104                          strncat(master_list, " ", sizeof(master_list) - 1 - strnlen(master_list, sizeof(master_list)));                          strncat(master_list, " ", sizeof(master_list) - 1 - strnlen(master_list, sizeof(master_list)));
105                  }                  }
106                  mysql_free_result(rs2);                  mysql_free_result(rs2);
107                    rs2 = NULL;
108    
109                  p_section = section_list_find_by_sid(sid);                  p_section = section_list_find_by_sid(sid);
110    
# Line 131  int load_section_config_from_db(void) Line 134  int load_section_config_from_db(void)
134                                  break;                                  break;
135                          }                          }
136    
137                          strncpy(p_section->sname, row[1], sizeof(p_section->sname) - 1);                          strncpy(p_section->sname, row[0], sizeof(p_section->sname) - 1);
138                          p_section->sname[sizeof(p_section->sname) - 1] = '\0';                          p_section->sname[sizeof(p_section->sname) - 1] = '\0';
139                          strncpy(p_section->stitle, row[1], sizeof(p_section->stitle) - 1);                          strncpy(p_section->stitle, row[1], sizeof(p_section->stitle) - 1);
140                          p_section->stitle[sizeof(p_section->stitle) - 1] = '\0';                          p_section->stitle[sizeof(p_section->stitle) - 1] = '\0';
# Line 144  int load_section_config_from_db(void) Line 147  int load_section_config_from_db(void)
147                  p_section->write_user_level = atoi(row[5]);                  p_section->write_user_level = atoi(row[5]);
148                  p_section->enable = (int8_t)atoi(row[6]);                  p_section->enable = (int8_t)atoi(row[6]);
149    
150                    // Update gen_ex menu set
151                    if (reload && p_section->enable && atoi(row[7]) > p_section->ex_menu_tm)
152                    {
153                            snprintf(ex_menu_conf, sizeof(ex_menu_conf), "%s/%d", VAR_GEN_EX_MENU_DIR, p_section->sid);
154    
155                            ret = load_menu(&ex_menu_set_new, ex_menu_conf);
156                            if (ret < 0)
157                            {
158                                    log_error("load_menu(%s) error: %d\n", ex_menu_conf, ret);
159                                    unload_menu(&ex_menu_set_new);
160                            }
161                            else
162                            {
163                                    if (p_section->ex_menu_tm > 0)
164                                    {
165                                            unload_menu(&(p_section->ex_menu_set));
166                                    }
167    
168                                    ex_menu_set_new.allow_exit = 1; // Allow exit menu
169                                    memcpy(&(p_section->ex_menu_set), &ex_menu_set_new, sizeof(ex_menu_set_new));
170                                    set_menu_shm_readonly(&(p_section->ex_menu_set));
171    
172                                    p_section->ex_menu_tm = atol(row[7]);
173    #ifdef _DEBUG
174                                    log_common("Loaded gen_ex_menu of section %d [%s]\n", p_section->sid, p_section->sname);
175    #endif
176                            }
177                    }
178    
179                  // release rw lock                  // release rw lock
180                  ret = section_list_rw_unlock(p_section);                  ret = section_list_rw_unlock(p_section);
181                  if (ret < 0)                  if (ret < 0)
# Line 151  int load_section_config_from_db(void) Line 183  int load_section_config_from_db(void)
183                          break;                          break;
184                  }                  }
185          }          }
         mysql_free_result(rs);  
186    
187    cleanup:
188            mysql_free_result(rs2);
189            mysql_free_result(rs);
190          mysql_close(db);          mysql_close(db);
191    
192          return ret;          return ret;
# Line 160  int load_section_config_from_db(void) Line 194  int load_section_config_from_db(void)
194    
195  int append_articles_from_db(int32_t start_aid, int global_lock, int article_count_limit)  int append_articles_from_db(int32_t start_aid, int global_lock, int article_count_limit)
196  {  {
197          MYSQL *db;          MYSQL *db = NULL;
198          MYSQL_RES *rs;          MYSQL_RES *rs = NULL;
199          MYSQL_ROW row;          MYSQL_ROW row;
200          char sql[SQL_BUFFER_LEN];          char sql[SQL_BUFFER_LEN];
201          ARTICLE article;          ARTICLE article;
# Line 177  int append_articles_from_db(int32_t star Line 211  int append_articles_from_db(int32_t star
211          if (db == NULL)          if (db == NULL)
212          {          {
213                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s\n", mysql_error(db));
214                  return -2;                  ret = -1;
215                    goto cleanup;
216          }          }
217    
218          snprintf(sql, sizeof(sql),          snprintf(sql, sizeof(sql),
# Line 191  int append_articles_from_db(int32_t star Line 226  int append_articles_from_db(int32_t star
226          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
227          {          {
228                  log_error("Query article list error: %s\n", mysql_error(db));                  log_error("Query article list error: %s\n", mysql_error(db));
229                  return -3;                  ret = -2;
230                    goto cleanup;
231          }          }
232          if ((rs = mysql_use_result(db)) == NULL)          if ((rs = mysql_use_result(db)) == NULL)
233          {          {
234                  log_error("Get article list data failed\n");                  log_error("Get article list data failed\n");
235                  return -3;                  ret = -2;
236                    goto cleanup;
237          }          }
238    
239          // acquire global lock          // acquire global lock
240          if (global_lock)          if (global_lock && (ret = section_list_rw_lock(NULL)) < 0)
241          {          {
242                  if ((ret = section_list_rw_lock(NULL)) < 0)                  log_error("section_list_rw_lock(sid = 0) error\n");
243                  {                  goto cleanup;
                         log_error("section_list_rw_lock(sid = 0) error\n");  
                         goto cleanup;  
                 }  
244          }          }
245    
246          while ((row = mysql_fetch_row(rs)))          while ((row = mysql_fetch_row(rs)))
247          {          {
248                  bzero(&article, sizeof(ARTICLE));                  memset(&article, 0, sizeof(ARTICLE));
249    
250                  // copy data of article                  // copy data of article
251                  i = 0;                  i = 0;
# Line 310  int append_articles_from_db(int32_t star Line 344  int append_articles_from_db(int32_t star
344          }          }
345    
346          // release global lock          // release global lock
347          if (global_lock)          if (global_lock && (ret = section_list_rw_unlock(NULL)) < 0)
348          {          {
349                  if ((ret = section_list_rw_unlock(NULL)) < 0)                  log_error("section_list_rw_unlock(sid=0) error\n");
                 {  
                         log_error("section_list_rw_unlock(sid=0) error\n");  
                 }  
350          }          }
351    
352  cleanup:  cleanup:
353          mysql_free_result(rs);          mysql_free_result(rs);
   
354          mysql_close(db);          mysql_close(db);
355    
356            article_cache_cleanup();
357    
358          return (ret < 0 ? ret : article_count);          return (ret < 0 ? ret : article_count);
359  }  }
360    
361  int set_last_article_op_log_from_db(void)  int set_last_article_op_log_from_db(void)
362  {  {
363          MYSQL *db;          MYSQL *db = NULL;
364          MYSQL_RES *rs;          MYSQL_RES *rs = NULL;
365          MYSQL_ROW row;          MYSQL_ROW row;
366          char sql[SQL_BUFFER_LEN];          char sql[SQL_BUFFER_LEN];
367            int ret = 0;
368    
369          db = db_open();          db = db_open();
370          if (db == NULL)          if (db == NULL)
371          {          {
372                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s\n", mysql_error(db));
373                  return -1;                  ret = -1;
374                    goto cleanup;
375          }          }
376    
377          snprintf(sql, sizeof(sql),          snprintf(sql, sizeof(sql),
# Line 346  int set_last_article_op_log_from_db(void Line 380  int set_last_article_op_log_from_db(void
380          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
381          {          {
382                  log_error("Query article op error: %s\n", mysql_error(db));                  log_error("Query article op error: %s\n", mysql_error(db));
383                  return -2;                  ret = -2;
384                    goto cleanup;
385          }          }
386          if ((rs = mysql_store_result(db)) == NULL)          if ((rs = mysql_store_result(db)) == NULL)
387          {          {
388                  log_error("Get article op data failed\n");                  log_error("Get article op data failed\n");
389                  return -2;                  ret = -2;
390                    goto cleanup;
391          }          }
392    
393          if ((row = mysql_fetch_row(rs)))          if ((row = mysql_fetch_row(rs)))
# Line 359  int set_last_article_op_log_from_db(void Line 395  int set_last_article_op_log_from_db(void
395                  last_article_op_log_mid = atoi(row[0]);                  last_article_op_log_mid = atoi(row[0]);
396          }          }
397    
398    cleanup:
399          mysql_free_result(rs);          mysql_free_result(rs);
   
400          mysql_close(db);          mysql_close(db);
401    
402          return last_article_op_log_mid;          return (ret < 0 ? ret : last_article_op_log_mid);
403  }  }
404    
405  int apply_article_op_log_from_db(int op_count_limit)  int apply_article_op_log_from_db(int op_count_limit)
406  {  {
407          MYSQL *db;          MYSQL *db = NULL;
408          MYSQL_RES *rs, *rs2;          MYSQL_RES *rs = NULL, *rs2 = NULL;
409          MYSQL_ROW row, row2;          MYSQL_ROW row, row2;
410          char sql[SQL_BUFFER_LEN];          char sql[SQL_BUFFER_LEN];
411          ARTICLE *p_article;          ARTICLE *p_article;
# Line 385  int apply_article_op_log_from_db(int op_ Line 421  int apply_article_op_log_from_db(int op_
421          if (db == NULL)          if (db == NULL)
422          {          {
423                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s\n", mysql_error(db));
424                  return -3;                  ret = -1;
425                    goto cleanup;
426          }          }
427    
428          snprintf(sql, sizeof(sql),          snprintf(sql, sizeof(sql),
# Line 397  int apply_article_op_log_from_db(int op_ Line 434  int apply_article_op_log_from_db(int op_
434          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
435          {          {
436                  log_error("Query article log error: %s\n", mysql_error(db));                  log_error("Query article log error: %s\n", mysql_error(db));
437                  return -3;                  ret = -2;
438                    goto cleanup;
439          }          }
440          if ((rs = mysql_store_result(db)) == NULL)          if ((rs = mysql_store_result(db)) == NULL)
441          {          {
442                  log_error("Get article log data failed\n");                  log_error("Get article log data failed\n");
443                  return -3;                  ret = -2;
444                    goto cleanup;
445          }          }
446    
447          while ((row = mysql_fetch_row(rs)))          while ((row = mysql_fetch_row(rs)))
# Line 514  int apply_article_op_log_from_db(int op_ Line 553  int apply_article_op_log_from_db(int op_
553                                  ret = -4;                                  ret = -4;
554                          }                          }
555                          mysql_free_result(rs2);                          mysql_free_result(rs2);
556                            rs2 = NULL;
557    
558                          p_article->excerption = 0; // Set excerption = 0 implicitly in case of rare condition                          p_article->excerption = 0; // Set excerption = 0 implicitly in case of rare condition
559                          break;                          break;
# Line 544  int apply_article_op_log_from_db(int op_ Line 584  int apply_article_op_log_from_db(int op_
584                                  ret = -4;                                  ret = -4;
585                          }                          }
586                          mysql_free_result(rs2);                          mysql_free_result(rs2);
587                            rs2 = NULL;
588    
589                          if (sid_dest > 0 && sid_dest != p_article->sid)                          if (sid_dest > 0 && sid_dest != p_article->sid)
590                          {                          {
# Line 580  int apply_article_op_log_from_db(int op_ Line 621  int apply_article_op_log_from_db(int op_
621                          p_article->excerption = 0;                          p_article->excerption = 0;
622                          break;                          break;
623                  case 'F': // Set article on top                  case 'F': // Set article on top
                         p_article->ontop = 1;  
                         break;  
624                  case 'V': // Unset article on top                  case 'V': // Unset article on top
625                          p_article->ontop = 0;                          p_article->ontop = (row[2][0] == 'F' ? 1 : 0);
626                            if (section_list_update_article_ontop(p_section, p_article) < 0)
627                            {
628                                    log_error("section_list_update_article_ontop(sid=%d, aid=%d) error\n",
629                                                      p_section->sid, p_article->aid);
630                            }
631                          break;                          break;
632                  case 'Z': // Set article as trnasship                  case 'Z': // Set article as trnasship
633                          p_article->transship = 1;                          p_article->transship = 1;
# Line 613  int apply_article_op_log_from_db(int op_ Line 657  int apply_article_op_log_from_db(int op_
657                  }                  }
658          }          }
659    
660    cleanup:
661            mysql_free_result(rs2);
662          mysql_free_result(rs);          mysql_free_result(rs);
   
663          mysql_close(db);          mysql_close(db);
664    
665          return (ret < 0 ? ret : op_count);          return (ret < 0 ? ret : op_count);
# Line 622  int apply_article_op_log_from_db(int op_ Line 667  int apply_article_op_log_from_db(int op_
667    
668  int section_list_loader_launch(void)  int section_list_loader_launch(void)
669  {  {
670            struct sigaction act = {0};
671          int pid;          int pid;
672          int ret;          int ret;
673          int32_t last_aid;          int32_t last_aid;
# Line 659  int section_list_loader_launch(void) Line 705  int section_list_loader_launch(void)
705          free(p_bbs_menu);          free(p_bbs_menu);
706          p_bbs_menu = NULL;          p_bbs_menu = NULL;
707    
708            // Set signal handler
709            act.sa_handler = SIG_DFL;
710            if (sigaction(SIGCHLD, &act, NULL) == -1)
711            {
712                    log_error("set signal action of SIGCHLD error: %d\n", errno);
713            }
714    
715            // Force reload to load gen_ex_menu
716            SYS_section_list_reload = 1;
717    
718          // Do section data loader periodically          // Do section data loader periodically
719          while (!SYS_server_exit)          while (!SYS_server_exit)
720          {          {
# Line 667  int section_list_loader_launch(void) Line 723  int section_list_loader_launch(void)
723                          SYS_section_list_reload = 0;                          SYS_section_list_reload = 0;
724    
725                          // Load section config                          // Load section config
726                          if (load_section_config_from_db() < 0)                          if (load_section_config_from_db(1) < 0)
727                          {                          {
728                                  log_error("load_section_config_from_db() error\n");                                  log_error("load_section_config_from_db() error\n");
729                          }                          }
730                          else                          else
731                          {                          {
732                                  log_error("Reload section config successfully\n");                                  log_common("Reload section config successfully\n");
733                          }                          }
734                  }                  }
735    
# Line 740  int section_list_loader_launch(void) Line 796  int section_list_loader_launch(void)
796    
797          // Child process exit          // Child process exit
798    
799            article_cache_cleanup();
800    
801            // gen_ex_menu cleanup
802            section_list_ex_menu_set_cleanup();
803            
804          // Detach data pools shm          // Detach data pools shm
805          detach_section_list_shm();          detach_section_list_shm();
806          detach_article_block_shm();          detach_article_block_shm();
# Line 772  int section_list_loader_reload(void) Line 833  int section_list_loader_reload(void)
833          return 0;          return 0;
834  }  }
835    
836  int query_section_articles(SECTION_LIST *p_section, int page_id, ARTICLE *p_articles[], int *p_article_count, int *p_page_count)  int query_section_articles(SECTION_LIST *p_section, int page_id, ARTICLE *p_articles[],
837                                                       int *p_article_count, int *p_page_count, int *p_ontop_start_offset)
838  {  {
839          ARTICLE *p_article;          ARTICLE *p_article;
840          ARTICLE *p_next_page_first_article;          ARTICLE *p_next_page_first_article;
841          int ret = 0;          int ret = 0;
842            int i;
843    
844          if (p_section == NULL || p_articles == NULL || p_article_count == NULL || p_page_count == NULL)          if (p_section == NULL || p_articles == NULL || p_article_count == NULL || p_page_count == NULL || p_ontop_start_offset == NULL)
845          {          {
846                  log_error("query_section_articles() NULL pointer error\n");                  log_error("NULL pointer error\n");
847                  return -1;                  return -1;
848          }          }
849    
# Line 791  int query_section_articles(SECTION_LIST Line 854  int query_section_articles(SECTION_LIST
854                  return -2;                  return -2;
855          }          }
856    
857          *p_page_count = p_section->page_count;          *p_page_count = section_list_page_count_with_ontop(p_section);
858            *p_article_count = 0;
859            *p_ontop_start_offset = BBS_article_limit_per_page;
860    
861          if (p_section->visible_article_count == 0)          if (p_section->visible_article_count == 0)
862          {          {
863                  *p_article_count = 0;                  // empty section
864          }          }
865          else if (page_id < 0 || page_id >= p_section->page_count)          else if (page_id < 0 || page_id >= *p_page_count)
866          {          {
867                  log_error("Invalid page_id=%d, not in range [0, %d)\n", page_id, p_section->page_count);                  log_error("Invalid page_id=%d, not in range [0, %d)\n", page_id, *p_page_count);
868                  ret = -3;                  ret = -3;
869          }          }
870          else          else
871          {          {
872                  ret = page_id;                  ret = page_id;
                 p_article = p_section->p_page_first_article[page_id];  
                 p_next_page_first_article =  
                         (page_id == p_section->page_count - 1 ? p_section->p_article_head : p_section->p_page_first_article[page_id + 1]);  
                 *p_article_count = 0;  
873    
874                  do                  if (page_id <= p_section->page_count - 1)
875                  {                  {
876                          if (p_article->visible)                          p_article = p_section->p_page_first_article[page_id];
877                            p_next_page_first_article =
878                                    (page_id == p_section->page_count - 1 ? p_section->p_article_head : p_section->p_page_first_article[page_id + 1]);
879    
880                            do
881                            {
882                                    if (p_article->visible)
883                                    {
884                                            p_articles[*p_article_count] = p_article;
885                                            (*p_article_count)++;
886                                    }
887                                    p_article = p_article->p_next;
888                            } while (p_article != p_next_page_first_article && (*p_article_count) <= BBS_article_limit_per_page);
889    
890                            if (*p_article_count != (page_id < p_section->page_count - 1 ? BBS_article_limit_per_page : p_section->last_page_visible_article_count))
891                          {                          {
892                                  p_articles[*p_article_count] = p_article;                                  log_error("Inconsistent visible article count %d detected in section %d page %d\n", *p_article_count, p_section->sid, page_id);
                                 (*p_article_count)++;  
893                          }                          }
894                          p_article = p_article->p_next;                  }
                 } while (p_article != p_next_page_first_article && (*p_article_count) <= BBS_article_limit_per_page);  
895    
896                  if (*p_article_count != (page_id < p_section->page_count - 1 ? BBS_article_limit_per_page : p_section->last_page_visible_article_count))                  *p_ontop_start_offset = *p_article_count;
897    
898                    // Append ontop articles
899                    if (page_id >= p_section->page_count - 1)
900                  {                  {
901                          log_error("Inconsistent visible article count %d detected in section %d page %d\n", *p_article_count, p_section->sid, page_id);                          i = (page_id == p_section->page_count - 1
902                                             ? 0
903                                             : (page_id - p_section->page_count + 1) * BBS_article_limit_per_page - p_section->last_page_visible_article_count);
904                            while (*p_article_count < BBS_article_limit_per_page && i < p_section->ontop_article_count)
905                            {
906                                    p_articles[(*p_article_count)++] = p_section->p_ontop_articles[i++];
907                            }
908                  }                  }
909          }          }
910    
# Line 836  int query_section_articles(SECTION_LIST Line 918  int query_section_articles(SECTION_LIST
918          return ret;          return ret;
919  }  }
920    
921  int locate_article_in_section(SECTION_LIST *p_section, const ARTICLE *p_article_cur, int direction,  int locate_article_in_section(SECTION_LIST *p_section, const ARTICLE *p_article_cur, int direction, int step,
922                                                            int *p_page_id, int *p_visible_offset, int *p_article_count)                                                            int *p_page_id, int *p_visible_offset, int *p_article_count)
923  {  {
924          const ARTICLE *p_article = NULL;          const ARTICLE *p_article = NULL;
925          ARTICLE *p_tmp;          ARTICLE *p_tmp;
926          int32_t aid = 0;          int32_t aid = 0;
927          int page_id;          int page_id = 0;
928          int offset;          int offset = 0;
929          int ret = 0;          int ret = 0;
930          int i;          int i;
931    
932          if (p_section == NULL || p_article_cur == NULL || p_page_id == NULL || p_visible_offset == NULL || p_article_count == NULL)          if (p_section == NULL || p_article_cur == NULL || p_page_id == NULL || p_visible_offset == NULL || p_article_count == NULL)
933          {          {
934                  log_error("locate_article_in_section() NULL pointer error\n");                  log_error("NULL pointer error\n");
935                  return -1;                  return -1;
936          }          }
937    
# Line 866  int locate_article_in_section(SECTION_LI Line 948  int locate_article_in_section(SECTION_LI
948          }          }
949          else if (direction == 1)          else if (direction == 1)
950          {          {
951                  p_article = p_article_cur;                  for (p_article = p_article_cur; step > 0 && p_article->p_topic_next->aid > p_article_cur->aid; p_article = p_article->p_topic_next)
                 do  
952                  {                  {
953                          p_article = p_article->p_topic_next;                          if (p_article->visible)
954                  } while (p_article != p_article_cur && p_article->visible == 0);                          {
955                                    step--;
956                            }
957                    }
958    
959                  aid = (p_article->aid > p_article_cur->aid ? p_article->aid : 0);                  aid = (p_article->aid > p_article_cur->aid && p_article->visible ? p_article->aid : 0);
960          }          }
961          else if (direction == -1)          else if (direction == -1)
962          {          {
963                  p_article = p_article_cur;                  for (p_article = p_article_cur; step > 0 && p_article->p_topic_prior->aid < p_article_cur->aid; p_article = p_article->p_topic_prior)
                 do  
964                  {                  {
965                          p_article = p_article->p_topic_prior;                          if (p_article->visible)
966                  } while (p_article != p_article_cur && p_article->visible == 0);                          {
967                                    step--;
968                            }
969                    }
970    
971                  aid = (p_article->aid < p_article_cur->aid ? p_article->aid : 0);                  aid = (p_article->aid < p_article_cur->aid && p_article->visible ? p_article->aid : 0);
972          }          }
973    
974          p_article = NULL;          p_article = NULL;
# Line 895  int locate_article_in_section(SECTION_LI Line 981  int locate_article_in_section(SECTION_LI
981                          *p_article_count = (page_id == p_section->page_count - 1 ? p_section->last_page_visible_article_count : BBS_article_limit_per_page);                          *p_article_count = (page_id == p_section->page_count - 1 ? p_section->last_page_visible_article_count : BBS_article_limit_per_page);
982    
983                          p_article = p_section->p_page_first_article[page_id];                          p_article = p_section->p_page_first_article[page_id];
984                          for (i = 0; i < *p_article_count;)                          for (i = 0; i < *p_article_count && p_article != NULL;)
985                          {                          {
986                                  if (p_article->visible)                                  if (p_article->visible)
987                                  {                                  {
# Line 910  int locate_article_in_section(SECTION_LI Line 996  int locate_article_in_section(SECTION_LI
996                                          {                                          {
997                                                  log_error("Visible article (aid=%d) not found in page %d\n", aid, page_id);                                                  log_error("Visible article (aid=%d) not found in page %d\n", aid, page_id);
998                                                  p_article = NULL;                                                  p_article = NULL;
999                                                    break;
1000                                          }                                          }
1001                                  }                                  }
1002                                  p_article = p_article->p_next;                                  p_article = p_article->p_next;
1003                          }                          }
1004    
1005                            // Include ontop articles
1006                            *p_article_count = section_list_page_article_count_with_ontop(p_section, page_id);
1007                  }                  }
1008          }          }
1009    
# Line 926  int locate_article_in_section(SECTION_LI Line 1016  int locate_article_in_section(SECTION_LI
1016    
1017          return (ret < 0 ? ret : (p_article == NULL ? 0 : 1));          return (ret < 0 ? ret : (p_article == NULL ? 0 : 1));
1018  }  }
1019    
1020    int get_section_ex_menu_set(SECTION_LIST *p_section, MENU_SET *p_ex_menu_set)
1021    {
1022            int ret = 0;
1023    
1024            if (p_section == NULL || p_ex_menu_set == NULL)
1025            {
1026                    log_error("NULL pointer error\n");
1027                    return -1;
1028            }
1029    
1030            // acquire lock of section
1031            if ((ret = section_list_rd_lock(p_section)) < 0)
1032            {
1033                    log_error("section_list_rd_lock(sid = %d) error\n", p_section->sid);
1034                    return -2;
1035            }
1036    
1037            memcpy(p_ex_menu_set, &(p_section->ex_menu_set), sizeof(p_section->ex_menu_set));
1038    
1039            // release lock of section
1040            if (section_list_rd_unlock(p_section) < 0)
1041            {
1042                    log_error("section_list_rd_unlock(sid = %d) error\n", p_section->sid);
1043                    ret = -2;
1044            }
1045    
1046            return ret;
1047    }

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