/[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.47 by sysadm, Wed Oct 8 03:34:40 2025 UTC Revision 1.72 by sysadm, Tue Jan 20 07:48:09 2026 UTC
# Line 1  Line 1 
1  /***************************************************************************  /* SPDX-License-Identifier: GPL-3.0-or-later */
2                                          section_list_loader.c  -  description  /*
3                                                           -------------------   * section_list_loader
4          Copyright            : (C) 2004-2025 by Leaflet   *   - load and query operations of section articles
5          Email                : leaflet@leafok.com   *
6   ***************************************************************************/   * Copyright (C) 2004-2026  Leaflet <leaflet@leafok.com>
7     */
8  /***************************************************************************  
9   *                                                                         *  #ifdef HAVE_CONFIG_H
10   *   This program is free software; you can redistribute it and/or modify  *  #include "config.h"
11   *   it under the terms of the GNU General Public License as published by  *  #endif
  *   the Free Software Foundation; either version 3 of the License, or     *  
  *   (at your option) any later version.                                   *  
  *                                                                         *  
  ***************************************************************************/  
12    
13  #include "article_cache.h"  #include "article_cache.h"
14  #include "article_view_log.h"  #include "article_view_log.h"
# Line 22  Line 18 
18  #include "log.h"  #include "log.h"
19  #include "menu.h"  #include "menu.h"
20  #include "section_list_loader.h"  #include "section_list_loader.h"
21    #include "user_list.h"
22  #include "user_priv.h"  #include "user_priv.h"
23  #include <errno.h>  #include <errno.h>
24  #include <signal.h>  #include <signal.h>
# Line 33  Line 30 
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    static void loader_proc_sig_usr1_handler(int i)
34    {
35            // Restart log
36            if (log_restart() < 0)
37            {
38                    log_error("Restart logging failed");
39            }
40    }
41    
42  int load_section_config_from_db(int update_gen_ex)  int load_section_config_from_db(int update_gen_ex)
43  {  {
44          MYSQL *db = NULL;          MYSQL *db = NULL;
# Line 43  int load_section_config_from_db(int upda Line 49  int load_section_config_from_db(int upda
49          char master_list[(BBS_username_max_len + 1) * 3 + 1];          char master_list[(BBS_username_max_len + 1) * 3 + 1];
50          SECTION_LIST *p_section;          SECTION_LIST *p_section;
51          char ex_menu_conf[FILE_PATH_LEN];          char ex_menu_conf[FILE_PATH_LEN];
         MENU_SET ex_menu_set_new;  
52          int ret = 0;          int ret = 0;
53    
54          db = db_open();          db = db_open();
55          if (db == NULL)          if (db == NULL)
56          {          {
57                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s", mysql_error(db));
58                  ret = -1;                  ret = -1;
59                  goto cleanup;                  goto cleanup;
60          }          }
# Line 63  int load_section_config_from_db(int upda Line 68  int load_section_config_from_db(int upda
68    
69          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
70          {          {
71                  log_error("Query section_list error: %s\n", mysql_error(db));                  log_error("Query section_list error: %s", mysql_error(db));
72                  ret = -2;                  ret = -2;
73                  goto cleanup;                  goto cleanup;
74          }          }
75          if ((rs = mysql_store_result(db)) == NULL)          if ((rs = mysql_store_result(db)) == NULL)
76          {          {
77                  log_error("Get section_list data failed\n");                  log_error("Get section_list data failed");
78                  ret = -2;                  ret = -2;
79                  goto cleanup;                  goto cleanup;
80          }          }
# Line 89  int load_section_config_from_db(int upda Line 94  int load_section_config_from_db(int upda
94    
95                  if (mysql_query(db, sql) != 0)                  if (mysql_query(db, sql) != 0)
96                  {                  {
97                          log_error("Query section_master error: %s\n", mysql_error(db));                          log_error("Query section_master error: %s", mysql_error(db));
98                          ret = -3;                          ret = -3;
99                          break;                          break;
100                  }                  }
101                  if ((rs2 = mysql_store_result(db)) == NULL)                  if ((rs2 = mysql_store_result(db)) == NULL)
102                  {                  {
103                          log_error("Get section_master data failed\n");                          log_error("Get section_master data failed");
104                          ret = -3;                          ret = -3;
105                          break;                          break;
106                  }                  }
# Line 116  int load_section_config_from_db(int upda Line 121  int load_section_config_from_db(int upda
121                          p_section = section_list_create(sid, row[1], row[2], master_list);                          p_section = section_list_create(sid, row[1], row[2], master_list);
122                          if (p_section == NULL)                          if (p_section == NULL)
123                          {                          {
124                                  log_error("section_list_create() error: load new section sid = %d sname = %s\n", sid, row[1]);                                  log_error("section_list_create() error: load new section sid = %d sname = %s", sid, row[1]);
125                                  ret = -4;                                  ret = -4;
126                                  break;                                  break;
127                          }                          }
# Line 137  int load_section_config_from_db(int upda Line 142  int load_section_config_from_db(int upda
142                                  break;                                  break;
143                          }                          }
144    
145                          strncpy(p_section->sname, row[1], sizeof(p_section->sname) - 1);                          if (section_list_update(p_section, row[1], row[2], master_list) < 0)
146                          p_section->sname[sizeof(p_section->sname) - 1] = '\0';                          {
147                          strncpy(p_section->stitle, row[2], sizeof(p_section->stitle) - 1);                                  log_error("section_list_update(sid=%d) error", p_section->sid);
148                          p_section->stitle[sizeof(p_section->stitle) - 1] = '\0';                                  ret = -4;
149                          strncpy(p_section->master_list, master_list, sizeof(p_section->master_list) - 1);                                  break;
150                          p_section->master_list[sizeof(p_section->master_list) - 1] = '\0';                          }
151                  }                  }
152    
153                  p_section->class_id = atoi(row[3]);                  p_section->class_id = atoi(row[3]);
# Line 155  int load_section_config_from_db(int upda Line 160  int load_section_config_from_db(int upda
160                  {                  {
161                          snprintf(ex_menu_conf, sizeof(ex_menu_conf), "%s/%d", VAR_GEN_EX_MENU_DIR, p_section->sid);                          snprintf(ex_menu_conf, sizeof(ex_menu_conf), "%s/%d", VAR_GEN_EX_MENU_DIR, p_section->sid);
162    
163                          ret = load_menu(&ex_menu_set_new, ex_menu_conf);                          if (detach_menu_shm(&(p_section->ex_menu_set)) < 0)
164                          if (ret < 0)                          {
165                                    log_error("detach_menu_shm(%s) error", ex_menu_conf);
166                            }
167                            if (load_menu(&(p_section->ex_menu_set), ex_menu_conf) < 0)
168                          {                          {
169                                  unload_menu(&ex_menu_set_new);                                  log_error("load_menu(%s) error", ex_menu_conf);
170                                  log_error("load_menu(%s) error: %d\n", ex_menu_conf, ret);                                  unload_menu(&(p_section->ex_menu_set));
171                          }                          }
172                          else                          else
173                          {                          {
174                                  if (p_section->ex_menu_tm > 0)                                  p_section->ex_menu_set.allow_exit = 1; // Allow exit menu
                                 {  
                                         unload_menu(&(p_section->ex_menu_set));  
                                 }  
   
                                 ex_menu_set_new.allow_exit = 1; // Allow exit menu  
                                 memcpy(&(p_section->ex_menu_set), &ex_menu_set_new, sizeof(ex_menu_set_new));  
   
175                                  p_section->ex_menu_tm = atol(row[7]);                                  p_section->ex_menu_tm = atol(row[7]);
176  #ifdef _DEBUG                                  log_debug("Loaded gen_ex_menu of section %d [%s]", p_section->sid, p_section->sname);
                                 log_common("Loaded gen_ex_menu of section %d [%s]\n", p_section->sid, p_section->sname);  
 #endif  
177                          }                          }
178                  }                  }
179    
# Line 212  int append_articles_from_db(int32_t star Line 211  int append_articles_from_db(int32_t star
211          db = db_open();          db = db_open();
212          if (db == NULL)          if (db == NULL)
213          {          {
214                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s", mysql_error(db));
215                  ret = -1;                  ret = -1;
216                  goto cleanup;                  goto cleanup;
217          }          }
# Line 227  int append_articles_from_db(int32_t star Line 226  int append_articles_from_db(int32_t star
226    
227          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
228          {          {
229                  log_error("Query article list error: %s\n", mysql_error(db));                  log_error("Query article list error: %s", mysql_error(db));
230                  ret = -2;                  ret = -2;
231                  goto cleanup;                  goto cleanup;
232          }          }
233          if ((rs = mysql_use_result(db)) == NULL)          if ((rs = mysql_use_result(db)) == NULL)
234          {          {
235                  log_error("Get article list data failed\n");                  log_error("Get article list data failed");
236                  ret = -2;                  ret = -2;
237                  goto cleanup;                  goto cleanup;
238          }          }
# Line 241  int append_articles_from_db(int32_t star Line 240  int append_articles_from_db(int32_t star
240          // acquire global lock          // acquire global lock
241          if (global_lock && (ret = section_list_rw_lock(NULL)) < 0)          if (global_lock && (ret = section_list_rw_lock(NULL)) < 0)
242          {          {
243                  log_error("section_list_rw_lock(sid = 0) error\n");                  log_error("section_list_rw_lock(sid = 0) error");
244                  goto cleanup;                  goto cleanup;
245          }          }
246    
# Line 281  int append_articles_from_db(int32_t star Line 280  int append_articles_from_db(int32_t star
280                  {                  {
281                          if ((ret = section_list_rw_unlock(p_section)) < 0)                          if ((ret = section_list_rw_unlock(p_section)) < 0)
282                          {                          {
283                                  log_error("section_list_rw_unlock(sid=%d) error\n", p_section->sid);                                  log_error("section_list_rw_unlock(sid=%d) error", p_section->sid);
284                                  break;                                  break;
285                          }                          }
286                  }                  }
287    
288                  if ((p_section = section_list_find_by_sid(article.sid)) == NULL)                  if ((p_section = section_list_find_by_sid(article.sid)) == NULL)
289                  {                  {
290                          log_error("section_list_find_by_sid(%d) error: unknown section, try reloading section config\n", article.sid);                          log_error("section_list_find_by_sid(%d) error: unknown section, try reloading section config", article.sid);
291                          ret = ERR_UNKNOWN_SECTION; // Unknown section found                          ret = ERR_UNKNOWN_SECTION; // Unknown section found
292                          break;                          break;
293                  }                  }
# Line 299  int append_articles_from_db(int32_t star Line 298  int append_articles_from_db(int32_t star
298                          p_topic = article_block_find_by_aid(article.tid);                          p_topic = article_block_find_by_aid(article.tid);
299                          if (p_topic == NULL || p_topic->visible == 0)                          if (p_topic == NULL || p_topic->visible == 0)
300                          {                          {
301                                  // log_error("Set article (aid = %d) as invisible due to invisible or non-existing topic head\n", article.aid);                                  // log_error("Set article (aid = %d) as invisible due to invisible or non-existing topic head", article.aid);
302                                  article.tid = 0;                                  article.tid = 0;
303                                  article.visible = 0;                                  article.visible = 0;
304                          }                          }
# Line 310  int append_articles_from_db(int32_t star Line 309  int append_articles_from_db(int32_t star
309                  {                  {
310                          if ((ret = section_list_rw_lock(p_section)) < 0)                          if ((ret = section_list_rw_lock(p_section)) < 0)
311                          {                          {
312                                  log_error("section_list_rw_lock(sid=0) error\n");                                  log_error("section_list_rw_lock(sid=0) error");
313                                  break;                                  break;
314                          }                          }
315                  }                  }
# Line 320  int append_articles_from_db(int32_t star Line 319  int append_articles_from_db(int32_t star
319    
320                  if (section_list_append_article(p_section, &article) < 0)                  if (section_list_append_article(p_section, &article) < 0)
321                  {                  {
322                          log_error("section_list_append_article(sid=%d, aid=%d) error\n",                          log_error("section_list_append_article(sid=%d, aid=%d) error",
323                                            p_section->sid, article.aid);                                            p_section->sid, article.aid);
324                          ret = -3;                          ret = -3;
325                          break;                          break;
# Line 330  int append_articles_from_db(int32_t star Line 329  int append_articles_from_db(int32_t star
329    
330                  if (article_cache_generate(VAR_ARTICLE_CACHE_DIR, &article, p_section, row[i++], sub_ip, 0) < 0)                  if (article_cache_generate(VAR_ARTICLE_CACHE_DIR, &article, p_section, row[i++], sub_ip, 0) < 0)
331                  {                  {
332                          log_error("article_cache_generate(aid=%d, cid=%d) error\n", article.aid, article.cid);                          log_error("article_cache_generate(aid=%d, cid=%d) error", article.aid, article.cid);
333                          ret = -4;                          ret = -4;
334                          break;                          break;
335                  }                  }
# Line 341  int append_articles_from_db(int32_t star Line 340  int append_articles_from_db(int32_t star
340          {          {
341                  if ((ret = section_list_rw_unlock(p_section)) < 0)                  if ((ret = section_list_rw_unlock(p_section)) < 0)
342                  {                  {
343                          log_error("section_list_rw_unlock(sid=%d) error\n", p_section->sid);                          log_error("section_list_rw_unlock(sid=%d) error", p_section->sid);
344                  }                  }
345          }          }
346    
347          // release global lock          // release global lock
348          if (global_lock && (ret = section_list_rw_unlock(NULL)) < 0)          if (global_lock && (ret = section_list_rw_unlock(NULL)) < 0)
349          {          {
350                  log_error("section_list_rw_unlock(sid=0) error\n");                  log_error("section_list_rw_unlock(sid=0) error");
351          }          }
352    
353  cleanup:  cleanup:
# Line 371  int set_last_article_op_log_from_db(void Line 370  int set_last_article_op_log_from_db(void
370          db = db_open();          db = db_open();
371          if (db == NULL)          if (db == NULL)
372          {          {
373                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s", mysql_error(db));
374                  ret = -1;                  ret = -1;
375                  goto cleanup;                  goto cleanup;
376          }          }
# Line 381  int set_last_article_op_log_from_db(void Line 380  int set_last_article_op_log_from_db(void
380    
381          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
382          {          {
383                  log_error("Query article op error: %s\n", mysql_error(db));                  log_error("Query article op error: %s", mysql_error(db));
384                  ret = -2;                  ret = -2;
385                  goto cleanup;                  goto cleanup;
386          }          }
387          if ((rs = mysql_store_result(db)) == NULL)          if ((rs = mysql_store_result(db)) == NULL)
388          {          {
389                  log_error("Get article op data failed\n");                  log_error("Get article op data failed");
390                  ret = -2;                  ret = -2;
391                  goto cleanup;                  goto cleanup;
392          }          }
# Line 422  int apply_article_op_log_from_db(int op_ Line 421  int apply_article_op_log_from_db(int op_
421          db = db_open();          db = db_open();
422          if (db == NULL)          if (db == NULL)
423          {          {
424                  log_error("db_open() error: %s\n", mysql_error(db));                  log_error("db_open() error: %s", mysql_error(db));
425                  ret = -1;                  ret = -1;
426                  goto cleanup;                  goto cleanup;
427          }          }
# Line 435  int apply_article_op_log_from_db(int op_ Line 434  int apply_article_op_log_from_db(int op_
434    
435          if (mysql_query(db, sql) != 0)          if (mysql_query(db, sql) != 0)
436          {          {
437                  log_error("Query article log error: %s\n", mysql_error(db));                  log_error("Query article log error: %s", mysql_error(db));
438                  ret = -2;                  ret = -2;
439                  goto cleanup;                  goto cleanup;
440          }          }
441          if ((rs = mysql_store_result(db)) == NULL)          if ((rs = mysql_store_result(db)) == NULL)
442          {          {
443                  log_error("Get article log data failed\n");                  log_error("Get article log data failed");
444                  ret = -2;                  ret = -2;
445                  goto cleanup;                  goto cleanup;
446          }          }
# Line 460  int apply_article_op_log_from_db(int op_ Line 459  int apply_article_op_log_from_db(int op_
459                  {                  {
460                          if ((ret = section_list_rw_unlock(p_section)) < 0)                          if ((ret = section_list_rw_unlock(p_section)) < 0)
461                          {                          {
462                                  log_error("section_list_rw_unlock(sid = %d) error\n", p_section->sid);                                  log_error("section_list_rw_unlock(sid = %d) error", p_section->sid);
463                                  break;                                  break;
464                          }                          }
465                  }                  }
466    
467                  if ((p_section = section_list_find_by_sid(p_article->sid)) == NULL)                  if ((p_section = section_list_find_by_sid(p_article->sid)) == NULL)
468                  {                  {
469                          log_error("section_list_find_by_sid(%d) error: unknown section, try reloading section config\n", p_article->sid);                          log_error("section_list_find_by_sid(%d) error: unknown section, try reloading section config", p_article->sid);
470                          ret = ERR_UNKNOWN_SECTION; // Unknown section found                          ret = ERR_UNKNOWN_SECTION; // Unknown section found
471                          break;                          break;
472                  }                  }
# Line 477  int apply_article_op_log_from_db(int op_ Line 476  int apply_article_op_log_from_db(int op_
476                  {                  {
477                          if ((ret = section_list_rw_lock(p_section)) < 0)                          if ((ret = section_list_rw_lock(p_section)) < 0)
478                          {                          {
479                                  log_error("section_list_rw_lock(sid = 0) error\n");                                  log_error("section_list_rw_lock(sid = 0) error");
480                                  break;                                  break;
481                          }                          }
482                  }                  }
# Line 487  int apply_article_op_log_from_db(int op_ Line 486  int apply_article_op_log_from_db(int op_
486                  switch (row[2][0])                  switch (row[2][0])
487                  {                  {
488                  case 'A': // Add article                  case 'A': // Add article
489                          log_error("Operation type=A should not be found\n");                          log_error("Operation type=A should not be found");
490                          break;                          break;
491                  case 'D': // Delete article                  case 'D': // Delete article
492                  case 'X': // Delete article by Admin                  case 'X': // Delete article by Admin
493                          if (section_list_set_article_visible(p_section, atoi(row[1]), 0) < 0)                          if (section_list_set_article_visible(p_section, atoi(row[1]), 0) < 0)
494                          {                          {
495                                  log_error("section_list_set_article_visible(sid=%d, aid=%d, visible=0) error\n", p_section->sid, atoi(row[1]));                                  log_error("section_list_set_article_visible(sid=%d, aid=%d, visible=0) error", p_section->sid, atoi(row[1]));
496                          }                          }
497                          if (section_list_calculate_page(p_section, atoi(row[1])) < 0)                          if (section_list_calculate_page(p_section, atoi(row[1])) < 0)
498                          {                          {
499                                  log_error("section_list_calculate_page(aid=%d) error\n", atoi(row[1]));                                  log_error("section_list_calculate_page(aid=%d) error", atoi(row[1]));
500                          }                          }
501                          break;                          break;
502                  case 'S': // Restore article                  case 'S': // Restore article
503                          if (section_list_set_article_visible(p_section, atoi(row[1]), 1) < 0)                          if (section_list_set_article_visible(p_section, atoi(row[1]), 1) < 0)
504                          {                          {
505                                  log_error("section_list_set_article_visible(sid=%d, aid=%d, visible=1) error\n", p_section->sid, atoi(row[1]));                                  log_error("section_list_set_article_visible(sid=%d, aid=%d, visible=1) error", p_section->sid, atoi(row[1]));
506                          }                          }
507                          if (section_list_calculate_page(p_section, atoi(row[1])) < 0)                          if (section_list_calculate_page(p_section, atoi(row[1])) < 0)
508                          {                          {
509                                  log_error("section_list_calculate_page(aid=%d) error\n", atoi(row[1]));                                  log_error("section_list_calculate_page(aid=%d) error", atoi(row[1]));
510                          }                          }
511                          break;                          break;
512                  case 'L': // Lock article                  case 'L': // Lock article
# Line 525  int apply_article_op_log_from_db(int op_ Line 524  int apply_article_op_log_from_db(int op_
524    
525                          if (mysql_query(db, sql) != 0)                          if (mysql_query(db, sql) != 0)
526                          {                          {
527                                  log_error("Query article error: %s\n", mysql_error(db));                                  log_error("Query article error: %s", mysql_error(db));
528                                  ret = -3;                                  ret = -3;
529                                  break;                                  break;
530                          }                          }
531                          if ((rs2 = mysql_use_result(db)) == NULL)                          if ((rs2 = mysql_use_result(db)) == NULL)
532                          {                          {
533                                  log_error("Get article data failed\n");                                  log_error("Get article data failed");
534                                  ret = -3;                                  ret = -3;
535                                  break;                                  break;
536                          }                          }
# Line 545  int apply_article_op_log_from_db(int op_ Line 544  int apply_article_op_log_from_db(int op_
544    
545                                  if (article_cache_generate(VAR_ARTICLE_CACHE_DIR, p_article, p_section, row2[2], sub_ip, 0) < 0)                                  if (article_cache_generate(VAR_ARTICLE_CACHE_DIR, p_article, p_section, row2[2], sub_ip, 0) < 0)
546                                  {                                  {
547                                          log_error("article_cache_generate(aid=%d, cid=%d) error\n", p_article->aid, p_article->cid);                                          log_error("article_cache_generate(aid=%d, cid=%d) error", p_article->aid, p_article->cid);
548                                          ret = -4;                                          ret = -4;
549                                  }                                  }
550                          }                          }
# Line 566  int apply_article_op_log_from_db(int op_ Line 565  int apply_article_op_log_from_db(int op_
565    
566                          if (mysql_query(db, sql) != 0)                          if (mysql_query(db, sql) != 0)
567                          {                          {
568                                  log_error("Query article error: %s\n", mysql_error(db));                                  log_error("Query article error: %s", mysql_error(db));
569                                  ret = -3;                                  ret = -3;
570                                  break;                                  break;
571                          }                          }
572                          if ((rs2 = mysql_store_result(db)) == NULL)                          if ((rs2 = mysql_store_result(db)) == NULL)
573                          {                          {
574                                  log_error("Get article data failed\n");                                  log_error("Get article data failed");
575                                  ret = -3;                                  ret = -3;
576                                  break;                                  break;
577                          }                          }
# Line 599  int apply_article_op_log_from_db(int op_ Line 598  int apply_article_op_log_from_db(int op_
598                                  // acquire lock of dest section                                  // acquire lock of dest section
599                                  if ((ret = section_list_rw_lock(p_section_dest)) < 0)                                  if ((ret = section_list_rw_lock(p_section_dest)) < 0)
600                                  {                                  {
601                                          log_error("section_list_rw_lock(sid = %d) error\n", p_section_dest);                                          log_error("section_list_rw_lock(sid = %d) error", p_section_dest);
602                                          break;                                          break;
603                                  }                                  }
604                                  // Move topic                                  // Move topic
605                                  if ((ret = section_list_move_topic(p_section, p_section_dest, p_article->aid)) < 0)                                  if ((ret = section_list_move_topic(p_section, p_section_dest, p_article->aid)) < 0)
606                                  {                                  {
607                                          log_error("section_list_move_topic(src_sid=%d, dest_sid=%d, aid=%d) error (%d), retry in the next loop\n",                                          log_error("section_list_move_topic(src_sid=%d, dest_sid=%d, aid=%d) error (%d), retry in the next loop",
608                                                            p_section->sid, p_section_dest->sid, p_article->aid, ret);                                                            p_section->sid, p_section_dest->sid, p_article->aid, ret);
609                                  }                                  }
610                                  // release lock of dest section                                  // release lock of dest section
611                                  if (section_list_rw_unlock(p_section_dest) < 0)                                  if (section_list_rw_unlock(p_section_dest) < 0)
612                                  {                                  {
613                                          log_error("section_list_rw_unlock(sid = %d) error\n", p_section_dest);                                          log_error("section_list_rw_unlock(sid = %d) error", p_section_dest);
614                                          ret = -1;                                          ret = -1;
615                                  }                                  }
616                          }                          }
# Line 627  int apply_article_op_log_from_db(int op_ Line 626  int apply_article_op_log_from_db(int op_
626                          p_article->ontop = (row[2][0] == 'F' ? 1 : 0);                          p_article->ontop = (row[2][0] == 'F' ? 1 : 0);
627                          if (section_list_update_article_ontop(p_section, p_article) < 0)                          if (section_list_update_article_ontop(p_section, p_article) < 0)
628                          {                          {
629                                  log_error("section_list_update_article_ontop(sid=%d, aid=%d) error\n",                                  log_error("section_list_update_article_ontop(sid=%d, aid=%d) error",
630                                                    p_section->sid, p_article->aid);                                                    p_section->sid, p_article->aid);
631                          }                          }
632                          break;                          break;
633                  case 'Z': // Set article as trnasship                  case 'Z': // Set article as transship
634                          p_article->transship = 1;                          p_article->transship = 1;
635                          break;                          break;
636                  default:                  default:
637                          // log_error("Operation type=%s unknown, mid=%s\n", row[2], row[0]);                          // log_error("Operation type=%s unknown, mid=%s", row[2], row[0]);
638                          break;                          break;
639                  }                  }
640    
# Line 655  int apply_article_op_log_from_db(int op_ Line 654  int apply_article_op_log_from_db(int op_
654          {          {
655                  if ((ret = section_list_rw_unlock(p_section)) < 0)                  if ((ret = section_list_rw_unlock(p_section)) < 0)
656                  {                  {
657                          log_error("section_list_rw_unlock(sid = %d) error\n", p_section->sid);                          log_error("section_list_rw_unlock(sid = %d) error", p_section->sid);
658                  }                  }
659          }          }
660    
# Line 689  int section_list_loader_launch(void) Line 688  int section_list_loader_launch(void)
688          int article_count;          int article_count;
689          int load_count;          int load_count;
690          int last_mid;          int last_mid;
691          int i;          time_t tm_section_list_reload = 0;
692            time_t tm_user_list_reload = time(NULL);
693            time_t tm_user_online_list_reload = time(NULL);
694    
695          if (section_list_loader_pid != 0)          if (section_list_loader_pid != 0)
696          {          {
697                  log_error("section_list_loader already running, pid = %d\n", section_list_loader_pid);                  log_error("section_list_loader already running, pid = %d", section_list_loader_pid);
698                  return -2;                  return -2;
699          }          }
700    
# Line 703  int section_list_loader_launch(void) Line 704  int section_list_loader_launch(void)
704          {          {
705                  SYS_child_process_count++;                  SYS_child_process_count++;
706                  section_list_loader_pid = pid;                  section_list_loader_pid = pid;
707                  log_common("Section list loader process (pid = %d) start\n", pid);                  log_common("Section list loader process (pid = %d) start", pid);
708                  return 0;                  return 0;
709          }          }
710          else if (pid < 0) // Error          else if (pid < 0) // Error
711          {          {
712                  log_error("fork() error (%d)\n", errno);                  log_error("fork() error (%d)", errno);
713                  return -1;                  return -1;
714          }          }
715    
# Line 717  int section_list_loader_launch(void) Line 718  int section_list_loader_launch(void)
718    
719          // Detach menu in shared memory          // Detach menu in shared memory
720          detach_menu_shm(&bbs_menu);          detach_menu_shm(&bbs_menu);
721            detach_menu_shm(&top10_menu);
722    
723          // Set signal handler          // Set signal handler
724          act.sa_handler = SIG_DFL;          act.sa_handler = SIG_IGN;
725          if (sigaction(SIGHUP, &act, NULL) == -1)          if (sigaction(SIGHUP, &act, NULL) == -1)
726          {          {
727                  log_error("set signal action of SIGHUP error: %d\n", errno);                  log_error("set signal action of SIGHUP error: %d", errno);
728          }          }
729          act.sa_handler = SIG_DFL;          act.sa_handler = SIG_DFL;
730          if (sigaction(SIGCHLD, &act, NULL) == -1)          if (sigaction(SIGCHLD, &act, NULL) == -1)
731          {          {
732                  log_error("set signal action of SIGCHLD error: %d\n", errno);                  log_error("set signal action of SIGCHLD error: %d", errno);
733            }
734            act.sa_handler = loader_proc_sig_usr1_handler;
735            if (sigaction(SIGUSR1, &act, NULL) == -1)
736            {
737                    log_error("set signal action of SIGUSR1 error: %d", errno);
738          }          }
739    
740          // Do section data loader periodically          // Do section data loader periodically
741          while (!SYS_server_exit)          while (!SYS_server_exit)
742          {          {
743                    tm_section_list_reload = time(NULL);
744    
745                  if (SYS_conf_reload)                  if (SYS_conf_reload)
746                  {                  {
747                          SYS_conf_reload = 0;                          SYS_conf_reload = 0;
# Line 740  int section_list_loader_launch(void) Line 749  int section_list_loader_launch(void)
749                          // Load section config                          // Load section config
750                          if (load_section_config_from_db(0) < 0)                          if (load_section_config_from_db(0) < 0)
751                          {                          {
752                                  log_error("load_section_config_from_db(0) error\n");                                  log_error("load_section_config_from_db(0) error");
753                          }                          }
754                          else                          else
755                          {                          {
756                                  log_common("Reload section config successfully\n");                                  log_common("Reload section config successfully");
757                          }                          }
758                  }                  }
759    
# Line 757  int section_list_loader_launch(void) Line 766  int section_list_loader_launch(void)
766    
767                          if ((ret = append_articles_from_db(last_aid + 1, 0, LOAD_ARTICLE_COUNT_LIMIT)) < 0)                          if ((ret = append_articles_from_db(last_aid + 1, 0, LOAD_ARTICLE_COUNT_LIMIT)) < 0)
768                          {                          {
769                                  log_error("append_articles_from_db(%d, 0, %d) error\n", last_aid + 1, LOAD_ARTICLE_COUNT_LIMIT);                                  log_error("append_articles_from_db(%d, 0, %d) error", last_aid + 1, LOAD_ARTICLE_COUNT_LIMIT);
770    
771                                  if (ret == ERR_UNKNOWN_SECTION)                                  if (ret == ERR_UNKNOWN_SECTION)
772                                  {                                  {
# Line 769  int section_list_loader_launch(void) Line 778  int section_list_loader_launch(void)
778                  load_count = article_block_article_count() - article_count;                  load_count = article_block_article_count() - article_count;
779                  if (load_count > 0)                  if (load_count > 0)
780                  {                  {
781                          log_common("Incrementally load %d articles, last_aid = %d\n", load_count, article_block_last_aid());                          log_common("Incrementally load %d articles, last_aid = %d", load_count, article_block_last_aid());
782                  }                  }
783    
784                  if (SYS_conf_reload)                  if (SYS_conf_reload)
# Line 784  int section_list_loader_launch(void) Line 793  int section_list_loader_launch(void)
793                  {                  {
794                          if ((ret = apply_article_op_log_from_db(LOAD_ARTICLE_COUNT_LIMIT)) < 0)                          if ((ret = apply_article_op_log_from_db(LOAD_ARTICLE_COUNT_LIMIT)) < 0)
795                          {                          {
796                                  log_error("apply_article_op_log_from_db() error\n");                                  log_error("apply_article_op_log_from_db() error");
797    
798                                  if (ret == ERR_UNKNOWN_SECTION)                                  if (ret == ERR_UNKNOWN_SECTION)
799                                  {                                  {
# Line 795  int section_list_loader_launch(void) Line 804  int section_list_loader_launch(void)
804    
805                  if (last_article_op_log_mid > last_mid)                  if (last_article_op_log_mid > last_mid)
806                  {                  {
807                          log_common("Proceeded %d article logs, last_mid = %d\n", last_article_op_log_mid - last_mid, last_article_op_log_mid);                          log_common("Proceeded %d article logs, last_mid = %d", last_article_op_log_mid - last_mid, last_article_op_log_mid);
808                  }                  }
809    
810                  if (SYS_conf_reload)                  // Reload user list
811                    if (time(NULL) - tm_user_list_reload >= BBS_user_list_load_interval)
812                  {                  {
813                          continue;                          if (user_list_pool_reload(0) < 0)
814                            {
815                                    log_error("user_list_pool_reload(all_user) error");
816                            }
817    
818                            if (user_stat_update() < 0)
819                            {
820                                    log_error("user_stat_update() error");
821                            }
822    
823                            tm_user_list_reload = time(NULL);
824                  }                  }
825    
826                  for (i = 0; i < BBS_section_list_load_interval && !SYS_server_exit && !SYS_conf_reload; i++)                  // Reload user online list
827                    if (time(NULL) - tm_user_online_list_reload >= BBS_user_online_list_load_interval)
828                    {
829                            if (user_list_pool_reload(1) < 0)
830                            {
831                                    log_error("user_list_pool_reload(online_user) error");
832                            }
833    
834                            tm_user_online_list_reload = time(NULL);
835                    }
836    
837                    // Wait for BBS_section_list_load_interval seconds
838                    while (!SYS_server_exit && time(NULL) - tm_section_list_reload < BBS_section_list_load_interval)
839                  {                  {
840                          sleep(1);                          sleep(1);
841                  }                  }
# Line 820  int section_list_loader_launch(void) Line 852  int section_list_loader_launch(void)
852          detach_section_list_shm();          detach_section_list_shm();
853          detach_article_block_shm();          detach_article_block_shm();
854          detach_trie_dict_shm();          detach_trie_dict_shm();
855            detach_user_list_pool_shm();
856    
857          log_common("Section list loader process exit normally\n");          log_common("Section list loader process exit normally");
858          log_end();          log_end();
859    
860          section_list_loader_pid = 0;          section_list_loader_pid = 0;
# Line 831  int section_list_loader_launch(void) Line 864  int section_list_loader_launch(void)
864          return 0;          return 0;
865  }  }
866    
867  int query_section_articles(SECTION_LIST *p_section, int page_id, ARTICLE *p_articles[],  int query_section_articles(SECTION_LIST *p_section, int page_id, const ARTICLE *p_articles[],
868                                                     int *p_article_count, int *p_page_count, int *p_ontop_start_offset)                                                     int *p_article_count, int *p_page_count, int *p_ontop_start_offset)
869  {  {
870          ARTICLE *p_article;          ARTICLE *p_article;
# Line 841  int query_section_articles(SECTION_LIST Line 874  int query_section_articles(SECTION_LIST
874    
875          if (p_section == NULL || p_articles == NULL || p_article_count == NULL || p_page_count == NULL || p_ontop_start_offset == NULL)          if (p_section == NULL || p_articles == NULL || p_article_count == NULL || p_page_count == NULL || p_ontop_start_offset == NULL)
876          {          {
877                  log_error("NULL pointer error\n");                  log_error("NULL pointer error");
878                  return -1;                  return -1;
879          }          }
880    
881          // acquire lock of section          // acquire lock of section
882          if ((ret = section_list_rd_lock(p_section)) < 0)          if ((ret = section_list_rd_lock(p_section)) < 0)
883          {          {
884                  log_error("section_list_rd_lock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
885                  return -2;                  return -2;
886          }          }
887    
# Line 862  int query_section_articles(SECTION_LIST Line 895  int query_section_articles(SECTION_LIST
895          }          }
896          else if (page_id < 0 || page_id >= *p_page_count)          else if (page_id < 0 || page_id >= *p_page_count)
897          {          {
898                  log_error("Invalid page_id=%d, not in range [0, %d)\n", page_id, *p_page_count);                  log_debug("Invalid page_id=%d, not in range [0, %d)", page_id, *p_page_count);
899                  ret = -3;                  ret = -3;
900          }          }
901          else          else
# Line 887  int query_section_articles(SECTION_LIST Line 920  int query_section_articles(SECTION_LIST
920    
921                          if (*p_article_count != (page_id < p_section->page_count - 1 ? BBS_article_limit_per_page : p_section->last_page_visible_article_count))                          if (*p_article_count != (page_id < p_section->page_count - 1 ? BBS_article_limit_per_page : p_section->last_page_visible_article_count))
922                          {                          {
923                                  log_error("Inconsistent visible article count %d detected in section %d page %d\n", *p_article_count, p_section->sid, page_id);                                  log_error("Inconsistent visible article count %d detected in section %d page %d", *p_article_count, p_section->sid, page_id);
924                          }                          }
925                  }                  }
926    
# Line 901  int query_section_articles(SECTION_LIST Line 934  int query_section_articles(SECTION_LIST
934                                           : (page_id - p_section->page_count + 1) * BBS_article_limit_per_page - p_section->last_page_visible_article_count);                                           : (page_id - p_section->page_count + 1) * BBS_article_limit_per_page - p_section->last_page_visible_article_count);
935                          while (*p_article_count < BBS_article_limit_per_page && i < p_section->ontop_article_count)                          while (*p_article_count < BBS_article_limit_per_page && i < p_section->ontop_article_count)
936                          {                          {
937                                  p_articles[(*p_article_count)++] = p_section->p_ontop_articles[i++];                                  p_articles[*p_article_count] = p_section->p_ontop_articles[i++];
938                                    (*p_article_count)++;
939                          }                          }
940                  }                  }
941          }          }
# Line 909  int query_section_articles(SECTION_LIST Line 943  int query_section_articles(SECTION_LIST
943          // release lock of section          // release lock of section
944          if (section_list_rd_unlock(p_section) < 0)          if (section_list_rd_unlock(p_section) < 0)
945          {          {
946                  log_error("section_list_rd_unlock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
947                  ret = -2;                  ret = -2;
948          }          }
949    
# Line 930  int locate_article_in_section(SECTION_LI Line 964  int locate_article_in_section(SECTION_LI
964    
965          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)
966          {          {
967                  log_error("NULL pointer error\n");                  log_error("NULL pointer error");
968                  return -1;                  return -1;
969          }          }
970    
971          // acquire lock of section          // acquire lock of section
972          if ((ret = section_list_rd_lock(p_section)) < 0)          if ((ret = section_list_rd_lock(p_section)) < 0)
973          {          {
974                  log_error("section_list_rd_lock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
975                  return -2;                  return -2;
976          }          }
977    
# Line 985  int locate_article_in_section(SECTION_LI Line 1019  int locate_article_in_section(SECTION_LI
1019                  p_article = section_list_find_article_with_offset(p_section, aid, &page_id, &offset, &p_tmp);                  p_article = section_list_find_article_with_offset(p_section, aid, &page_id, &offset, &p_tmp);
1020                  if (p_article != NULL)                  if (p_article != NULL)
1021                  {                  {
1022                          *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);
1023    
1024                          p_article = p_section->p_page_first_article[page_id];                          p_article = p_section->p_page_first_article[page_id];
1025                          for (i = 0; i < *p_article_count && p_article != NULL;)                          for (i = 0; i < *p_article_count && p_article != NULL;)
# Line 1001  int locate_article_in_section(SECTION_LI Line 1035  int locate_article_in_section(SECTION_LI
1035                                          i++;                                          i++;
1036                                          if (i >= *p_article_count)                                          if (i >= *p_article_count)
1037                                          {                                          {
1038                                                  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", aid, page_id);
1039                                                  p_article = NULL;                                                  p_article = NULL;
1040                                                  break;                                                  break;
1041                                          }                                          }
1042                                  }                                  }
1043    
1044                                  p_article = p_article->p_next;                                  p_article = p_article->p_next;
1045                                    if (p_article == p_section->p_page_first_article[page_id])
1046                                    {
1047                                            log_error("Dead loop detected at page=%d, article_count=%d", page_id, *p_article_count);
1048                                            break;
1049                                    }
1050                          }                          }
1051    
1052                          // Include ontop articles                          // Include ontop articles
# Line 1017  int locate_article_in_section(SECTION_LI Line 1057  int locate_article_in_section(SECTION_LI
1057          // release lock of section          // release lock of section
1058          if (section_list_rd_unlock(p_section) < 0)          if (section_list_rd_unlock(p_section) < 0)
1059          {          {
1060                  log_error("section_list_rd_unlock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
1061                  ret = -2;                  ret = -2;
1062          }          }
1063    
1064          return (ret < 0 ? ret : (p_article == NULL ? 0 : 1));          return (ret < 0 ? ret : (p_article == NULL ? 0 : 1));
1065  }  }
1066    
1067    int last_article_in_section(SECTION_LIST *p_section, const ARTICLE **pp_article)
1068    {
1069            int ret = 0;
1070    
1071            const ARTICLE *p_article;
1072    
1073            if (p_section == NULL || pp_article == NULL)
1074            {
1075                    log_error("NULL pointer error");
1076                    return -1;
1077            }
1078    
1079            *pp_article = NULL;
1080    
1081            // acquire lock of section
1082            if ((ret = section_list_rd_lock(p_section)) < 0)
1083            {
1084                    log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
1085                    return -2;
1086            }
1087    
1088            for (p_article = p_section->p_article_tail;
1089                     p_article && p_article != p_section->p_article_head && !p_article->visible;
1090                     p_article = p_article->p_prior)
1091                    ;
1092    
1093            if (p_article && p_article->visible)
1094            {
1095                    *pp_article = p_article;
1096                    ret = 1;
1097            }
1098    
1099            // release lock of section
1100            if (section_list_rd_unlock(p_section) < 0)
1101            {
1102                    log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
1103                    ret = -2;
1104            }
1105    
1106            return ret;
1107    }
1108    
1109  int scan_unread_article_in_section(SECTION_LIST *p_section, const ARTICLE *p_article_cur, const ARTICLE **pp_article_unread)  int scan_unread_article_in_section(SECTION_LIST *p_section, const ARTICLE *p_article_cur, const ARTICLE **pp_article_unread)
1110  {  {
1111          ARTICLE *p_article;          ARTICLE *p_article;
# Line 1031  int scan_unread_article_in_section(SECTI Line 1113  int scan_unread_article_in_section(SECTI
1113    
1114          if (p_section == NULL || p_article_cur == NULL || pp_article_unread == NULL)          if (p_section == NULL || p_article_cur == NULL || pp_article_unread == NULL)
1115          {          {
1116                  log_error("NULL pointer error\n");                  log_error("NULL pointer error");
1117                  return -1;                  return -1;
1118          }          }
1119    
1120          if (p_article_cur->sid != p_section->sid)          if (p_article_cur->sid != p_section->sid)
1121          {          {
1122                  log_error("Inconsistent SID\n");                  log_error("Inconsistent SID");
1123                  return -1;                  return -1;
1124          }          }
1125    
1126          // acquire lock of section          // acquire lock of section
1127          if ((ret = section_list_rd_lock(p_section)) < 0)          if ((ret = section_list_rd_lock(p_section)) < 0)
1128          {          {
1129                  log_error("section_list_rd_lock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
1130                  return -2;                  return -2;
1131          }          }
1132    
# Line 1062  int scan_unread_article_in_section(SECTI Line 1144  int scan_unread_article_in_section(SECTI
1144          // release lock of section          // release lock of section
1145          if (section_list_rd_unlock(p_section) < 0)          if (section_list_rd_unlock(p_section) < 0)
1146          {          {
1147                  log_error("section_list_rd_unlock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
1148                    return -2;
1149            }
1150    
1151            return (p_article != NULL && p_article != p_article_cur ? 1 : 0);
1152    }
1153    
1154    int scan_article_in_section_by_uid(SECTION_LIST *p_section, const ARTICLE *p_article_cur,
1155                                                                       int direction, int32_t uid, const ARTICLE **pp_article)
1156    {
1157            ARTICLE *p_article;
1158            int ret = 0;
1159    
1160            if (p_section == NULL || p_article_cur == NULL || pp_article == NULL)
1161            {
1162                    log_error("NULL pointer error");
1163                    return -1;
1164            }
1165    
1166            if (p_article_cur->sid != p_section->sid)
1167            {
1168                    log_error("Inconsistent SID");
1169                    return -1;
1170            }
1171    
1172            // acquire lock of section
1173            if ((ret = section_list_rd_lock(p_section)) < 0)
1174            {
1175                    log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
1176                    return -2;
1177            }
1178    
1179            *pp_article = NULL;
1180    
1181            if (direction >= 0)
1182            {
1183                    for (p_article = p_article_cur->p_next; p_article != NULL && p_article != p_article_cur; p_article = p_article->p_next)
1184                    {
1185                            if (p_article->visible && p_article->uid == uid)
1186                            {
1187                                    *pp_article = p_article;
1188                                    break;
1189                            }
1190                    }
1191            }
1192            else // direction < 0
1193            {
1194                    for (p_article = p_article_cur->p_prior; p_article != NULL && p_article != p_article_cur; p_article = p_article->p_prior)
1195                    {
1196                            if (p_article->visible && p_article->uid == uid)
1197                            {
1198                                    *pp_article = p_article;
1199                                    break;
1200                            }
1201                    }
1202            }
1203    
1204            // release lock of section
1205            if (section_list_rd_unlock(p_section) < 0)
1206            {
1207                    log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
1208                    return -2;
1209            }
1210    
1211            return (p_article != NULL && p_article != p_article_cur ? 1 : 0);
1212    }
1213    
1214    int scan_article_in_section_by_username(SECTION_LIST *p_section, const ARTICLE *p_article_cur,
1215                                                                                    int direction, const char *username, const ARTICLE **pp_article)
1216    {
1217            ARTICLE *p_article;
1218            int ret = 0;
1219    
1220            if (p_section == NULL || p_article_cur == NULL || username == NULL || pp_article == NULL)
1221            {
1222                    log_error("NULL pointer error");
1223                    return -1;
1224            }
1225    
1226            if (p_article_cur->sid != p_section->sid)
1227            {
1228                    log_error("Inconsistent SID");
1229                    return -1;
1230            }
1231    
1232            // acquire lock of section
1233            if ((ret = section_list_rd_lock(p_section)) < 0)
1234            {
1235                    log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
1236                    return -2;
1237            }
1238    
1239            *pp_article = NULL;
1240    
1241            if (direction >= 0)
1242            {
1243                    for (p_article = p_article_cur->p_next; p_article != NULL && p_article != p_article_cur; p_article = p_article->p_next)
1244                    {
1245                            if (p_article->visible && strcasecmp(p_article->username, username) == 0)
1246                            {
1247                                    *pp_article = p_article;
1248                                    break;
1249                            }
1250                    }
1251            }
1252            else // direction < 0
1253            {
1254                    for (p_article = p_article_cur->p_prior; p_article != NULL && p_article != p_article_cur; p_article = p_article->p_prior)
1255                    {
1256                            if (p_article->visible && strcasecmp(p_article->username, username) == 0)
1257                            {
1258                                    *pp_article = p_article;
1259                                    break;
1260                            }
1261                    }
1262            }
1263    
1264            // release lock of section
1265            if (section_list_rd_unlock(p_section) < 0)
1266            {
1267                    log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
1268                    return -2;
1269            }
1270    
1271            return (p_article != NULL && p_article != p_article_cur ? 1 : 0);
1272    }
1273    
1274    int scan_article_in_section_by_title(SECTION_LIST *p_section, const ARTICLE *p_article_cur,
1275                                                                             int direction, const char *title, const ARTICLE **pp_article)
1276    {
1277            ARTICLE *p_article;
1278            int ret = 0;
1279    
1280            if (p_section == NULL || p_article_cur == NULL || title == NULL || pp_article == NULL)
1281            {
1282                    log_error("NULL pointer error");
1283                    return -1;
1284            }
1285    
1286            if (p_article_cur->sid != p_section->sid)
1287            {
1288                    log_error("Inconsistent SID");
1289                    return -1;
1290            }
1291    
1292            // acquire lock of section
1293            if ((ret = section_list_rd_lock(p_section)) < 0)
1294            {
1295                    log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
1296                    return -2;
1297            }
1298    
1299            *pp_article = NULL;
1300    
1301            if (direction >= 0)
1302            {
1303                    for (p_article = p_article_cur->p_next; p_article != NULL && p_article != p_article_cur; p_article = p_article->p_next)
1304                    {
1305                            if (p_article->visible && strcasestr(p_article->title, title) != NULL)
1306                            {
1307                                    *pp_article = p_article;
1308                                    break;
1309                            }
1310                    }
1311            }
1312            else // direction < 0
1313            {
1314                    for (p_article = p_article_cur->p_prior; p_article != NULL && p_article != p_article_cur; p_article = p_article->p_prior)
1315                    {
1316                            if (p_article->visible && strcasestr(p_article->title, title) != NULL)
1317                            {
1318                                    *pp_article = p_article;
1319                                    break;
1320                            }
1321                    }
1322            }
1323    
1324            // release lock of section
1325            if (section_list_rd_unlock(p_section) < 0)
1326            {
1327                    log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
1328                  return -2;                  return -2;
1329          }          }
1330    
# Line 1075  int get_section_ex_menu_set(SECTION_LIST Line 1337  int get_section_ex_menu_set(SECTION_LIST
1337    
1338          if (p_section == NULL || p_ex_menu_set == NULL)          if (p_section == NULL || p_ex_menu_set == NULL)
1339          {          {
1340                  log_error("NULL pointer error\n");                  log_error("NULL pointer error");
1341                  return -1;                  return -1;
1342          }          }
1343    
1344          // acquire lock of section          // acquire lock of section
1345          if ((ret = section_list_rd_lock(p_section)) < 0)          if ((ret = section_list_rd_lock(p_section)) < 0)
1346          {          {
1347                  log_error("section_list_rd_lock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_lock(sid = %d) error", p_section->sid);
1348                  return -2;                  return -2;
1349          }          }
1350    
# Line 1091  int get_section_ex_menu_set(SECTION_LIST Line 1353  int get_section_ex_menu_set(SECTION_LIST
1353          // release lock of section          // release lock of section
1354          if (section_list_rd_unlock(p_section) < 0)          if (section_list_rd_unlock(p_section) < 0)
1355          {          {
1356                  log_error("section_list_rd_unlock(sid = %d) error\n", p_section->sid);                  log_error("section_list_rd_unlock(sid = %d) error", p_section->sid);
1357                  ret = -2;                  ret = -2;
1358          }          }
1359    

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