/[LeafOK_CVS]/lbbs/src/menu.c

Diff of /lbbs/src/menu.c

Parent Directory | Revision Log | View Patch Patch

- Revision 1.42 by sysadm,
 Thu May 15 08:53:23 2025 UTC
+ Revision 1.95 by sysadm,
 Mon Jan  5 05:41:34 2026 UTC
  Line 1
- /***************************************************************************
+ /* SPDX-License-Identifier: GPL-3.0-or-later */
-                                                   menu.c  -  description
+ /*
-                                                          -------------------
+  * menu
-         Copyright            : (C) 2004-2025 by Leaflet
+  *   - configurable user interactive menu feature
-         Email                : leaflet@leafok.com
+  *
-  ***************************************************************************/
+  * Copyright (C) 2004-2026  Leaflet <leaflet@leafok.com>
+  */
- /***************************************************************************
-  *                                                                         *
+ #ifdef HAVE_CONFIG_H
-  *   This program is free software; you can redistribute it and/or modify  *
+ #include "config.h"
-  *   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.                                   *
-  *                                                                         *
-  ***************************************************************************/
  #include "bbs.h"
  #include "bbs_cmd.h"
- #include "user_priv.h"
  #include "bbs_cmd.h"
- #include "menu.h"
+ #include "common.h"
- #include "log.h"
  #include "io.h"
+ #include "log.h"
+ #include "menu.h"
  #include "screen.h"
- #include "common.h"
+ #include "user_priv.h"
- #include <string.h>
- #include <stdio.h>
  #include <ctype.h>
- #include <stdlib.h>
  #include <errno.h>
+ #include <fcntl.h>
+ #include <stdio.h>
+ #include <stdlib.h>
+ #include <string.h>
  #include <unistd.h>
- #include <sys/shm.h>
+ #include <sys/mman.h>
- #include <sys/ipc.h>
+ #include <sys/stat.h>
- #define MENU_SCREEN_PATH_PREFIX "var/MENU_SCR_"
+ enum _menu_constant_t
- #define MENU_CONF_DELIM_WITH_SPACE " ,\t\r\n"
+ {
- #define MENU_CONF_DELIM_WITHOUT_SPACE "\r\n"
+         MENU_SET_RESERVED_LENGTH = sizeof(int16_t) * 4,
+ };
- #define MENU_SET_RESERVED_LENGTH (sizeof(int16_t) * 4)
+ static const char MENU_CONF_DELIM_WITH_SPACE[] = " ,\t\r\n";
+ static const char MENU_CONF_DELIM_WITHOUT_SPACE[] = "\r\n";
- MENU_SET *p_bbs_menu;
+ MENU_SET bbs_menu;
+ MENU_SET top10_menu;
  int load_menu(MENU_SET *p_menu_set, const char *conf_file)
  {
+         char filepath[FILE_PATH_LEN];
+         int fd;
+         size_t size;
+         void *p_shm;
          FILE *fin;
          int fin_line = 0;
          char buffer[LINE_BUFFER_LEN];
          char temp[LINE_BUFFER_LEN];
          char *p = NULL;
          char *q = NULL;
+         char *r = NULL;
          char *saveptr = NULL;
          MENU *p_menu = NULL;
          MENU_ITEM *p_menu_item = NULL;
- Line 55 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 60 
  int load_menu(MENU_SET *p_menu_set, cons
          MENU_ID menu_id;
          MENU_ITEM_ID menu_item_id;
          MENU_SCREEN_ID screen_id;
-         int proj_id;
-         key_t key;
+         if (p_menu_set == NULL || conf_file == NULL)
-         size_t size;
+         {
+                 log_error("NULL pointer error");
+                 return -1;
+         }
+         // Initialize the data structure
+         memset(p_menu_set, 0, sizeof(*p_menu_set));
          // Use trie_dict to search menu_id by menu name
          p_menu_set->p_menu_name_dict = trie_dict_create();
          if (p_menu_set->p_menu_name_dict == NULL)
          {
-                 log_error("trie_dict_create() error\n");
+                 log_error("trie_dict_create() error");
-                 return -3;
+                 return -1;
          }
          // Use trie_dict to search screen_id by menu screen name
          p_menu_set->p_menu_screen_dict = trie_dict_create();
          if (p_menu_set->p_menu_screen_dict == NULL)
          {
-                 log_error("trie_dict_create() error\n");
+                 log_error("trie_dict_create() error");
-                 return -3;
+                 return -1;
          }
          if ((fin = fopen(conf_file, "r")) == NULL)
          {
-                 log_error("Open %s failed", conf_file);
+                 log_error("Open %s error: %d", conf_file, errno);
                  return -2;
          }
          // Allocate shared memory
-         proj_id = (int)(time(NULL) % getpid());
-         key = ftok(conf_file, proj_id);
-         if (key == -1)
-         {
-                 log_error("ftok(%s %d) error (%d)\n", conf_file, proj_id, errno);
-                 return -2;
-         }
          size = MENU_SET_RESERVED_LENGTH +
                     sizeof(MENU) * MAX_MENUS +
                     sizeof(MENU_ITEM) * MAX_MENUITEMS +
                     sizeof(MENU_SCREEN) * MAX_MENUS +
                     MAX_MENU_SCR_BUF_LENGTH * MAX_MENUS;
-         p_menu_set->shmid = shmget(key, size, IPC_CREAT | IPC_EXCL | 0600);
-         if (p_menu_set->shmid == -1)
+         strncpy(filepath, conf_file, sizeof(filepath) - 1);
+         filepath[sizeof(filepath) - 1] = '\0';
+         snprintf(p_menu_set->shm_name, sizeof(p_menu_set->shm_name), "/MENU_SHM_%s", basename(filepath));
+         if (shm_unlink(p_menu_set->shm_name) == -1 && errno != ENOENT)
+         {
+                 log_error("shm_unlink(%s) error (%d)", p_menu_set->shm_name, errno);
+                 return -2;
+         }
+         if ((fd = shm_open(p_menu_set->shm_name, O_CREAT | O_EXCL | O_RDWR, 0600)) == -1)
+         {
+                 log_error("shm_open(%s) error (%d)", p_menu_set->shm_name, errno);
+                 return -2;
+         }
+         if (ftruncate(fd, (off_t)size) == -1)
+         {
+                 log_error("ftruncate(size=%d) error (%d)", size, errno);
+                 close(fd);
+                 return -2;
+         }
+         p_shm = mmap(NULL, size, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0L);
+         if (p_shm == MAP_FAILED)
          {
-                 log_error("shmget(size = %d) error (%d)\n", size, errno);
+                 log_error("mmap() error (%d)", errno);
-                 return -3;
+                 close(fd);
+                 return -2;
          }
-         p_menu_set->p_reserved = shmat(p_menu_set->shmid, NULL, 0);
-         if (p_menu_set->p_reserved == (void *)-1)
+         if (close(fd) < 0)
          {
-                 log_error("shmat() error (%d)\n", errno);
+                 log_error("close(fd) error (%d)", errno);
-                 return -3;
+                 return -1;
          }
-         p_menu_set->p_menu_pool = p_menu_set->p_reserved + MENU_SET_RESERVED_LENGTH;
-         p_menu_set->p_menu_item_pool = p_menu_set->p_menu_pool + sizeof(MENU) * MAX_MENUS;
+         p_menu_set->shm_size = size;
-         p_menu_set->p_menu_screen_pool = p_menu_set->p_menu_item_pool + sizeof(MENU_ITEM) * MAX_MENUITEMS;
+         p_menu_set->p_reserved = p_shm;
-         p_menu_set->p_menu_screen_buf = p_menu_set->p_menu_screen_pool + sizeof(MENU_SCREEN) * MAX_MENUS;
+         p_menu_set->p_menu_pool = (char *)(p_menu_set->p_reserved) + MENU_SET_RESERVED_LENGTH;
+         p_menu_set->p_menu_item_pool = (char *)(p_menu_set->p_menu_pool) + sizeof(MENU) * MAX_MENUS;
+         p_menu_set->p_menu_screen_pool = (char *)(p_menu_set->p_menu_item_pool) + sizeof(MENU_ITEM) * MAX_MENUITEMS;
+         p_menu_set->p_menu_screen_buf = (char *)(p_menu_set->p_menu_screen_pool) + sizeof(MENU_SCREEN) * MAX_MENUS;
          p_menu_set->p_menu_screen_buf_free = p_menu_set->p_menu_screen_buf;
          p_menu_set->menu_count = 0;
- Line 118 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 149 
  int load_menu(MENU_SET *p_menu_set, cons
          p_menu_set->menu_screen_count = 0;
          p_menu_set->choose_step = 0;
          p_menu_set->menu_id_path[0] = 0;
+         p_menu_set->menu_item_pos[0] = 0;
+         p_menu_set->allow_exit = 0;
          while (fgets(buffer, sizeof(buffer), fin))
          {
- Line 142 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 175 
  int load_menu(MENU_SET *p_menu_set, cons
                          {
                                  if (p_menu != NULL)
                                  {
-                                         log_error("Incomplete menu definition in menu config line %d\n", fin_line);
+                                         log_error("Incomplete menu definition in menu config line %d", fin_line);
                                          return -1;
                                  }
                                  if (p_menu_set->menu_count >= MAX_MENUS)
                                  {
-                                         log_error("Menu count (%d) exceed limit (%d)\n", p_menu_set->menu_count, MAX_MENUS);
+                                         log_error("Menu count (%d) exceed limit (%d)", p_menu_set->menu_count, MAX_MENUS);
                                          return -3;
                                  }
                                  menu_id = (MENU_ID)p_menu_set->menu_count;
- Line 159 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 192 
  int load_menu(MENU_SET *p_menu_set, cons
                                  p_menu->item_count = 0;
                                  p_menu->title.show = 0;
                                  p_menu->screen_show = 0;
+                                 p_menu->page_item_limit = 0;
+                                 p_menu->use_filter = 0;
+                                 p_menu->filter_handler = NULL;
                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                  if (q == NULL)
                                  {
-                                         log_error("Error menu name in menu config line %d\n", fin_line);
+                                         log_error("Error menu name in menu config line %d", fin_line);
                                          return -1;
                                  }
                                  p = q;
-                                 while (isalnum(*q) || *q == '_')
+                                 while (isalnum((int)*q) || *q == '_' || *q == '-')
                                  {
                                          q++;
                                  }
                                  if (*q != '\0')
                                  {
-                                         log_error("Error menu name in menu config line %d\n", fin_line);
+                                         log_error("Error menu name in menu config line %d", fin_line);
                                          return -1;
                                  }
                                  if (q - p > sizeof(p_menu->name) - 1)
                                  {
-                                         log_error("Too longer menu name in menu config line %d\n", fin_line);
+                                         log_error("Too longer menu name in menu config line %d", fin_line);
                                          return -1;
                                  }
                                  strncpy(p_menu->name, p, sizeof(p_menu->name) - 1);
- Line 187 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 223 
  int load_menu(MENU_SET *p_menu_set, cons
                                  if (trie_dict_set(p_menu_set->p_menu_name_dict, p_menu->name, (int64_t)menu_id) != 1)
                                  {
-                                         log_error("Error set menu dict [%s]\n", p_menu->name);
+                                         log_error("Error set menu dict [%s]", p_menu->name);
                                  }
                                  // Check syntax
                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                  if (q != NULL)
                                  {
-                                         log_error("Unknown extra content in menu config line %d\n", fin_line);
+                                         log_error("Unknown extra content in menu config line %d", fin_line);
                                          return -1;
                                  }
- Line 223 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 259 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  // BEGIN of menu item
                                                  if (p_menu->item_count >= MAX_ITEMS_PER_MENU)
                                                  {
-                                                         log_error("Menuitem count per menu (%d) exceed limit (%d)\n", p_menu->item_count, MAX_ITEMS_PER_MENU);
+                                                         log_error("Menuitem count per menu (%d) exceed limit (%d)", p_menu->item_count, MAX_ITEMS_PER_MENU);
                                                          return -1;
                                                  }
                                                  if (p_menu_set->menu_item_count >= MAX_MENUITEMS)
                                                  {
-                                                         log_error("Menuitem count (%d) exceed limit (%d)\n", p_menu_set->menu_item_count, MAX_MENUITEMS);
+                                                         log_error("Menuitem count (%d) exceed limit (%d)", p_menu_set->menu_item_count, MAX_MENUITEMS);
                                                          return -3;
                                                  }
                                                  menu_item_id = (MENU_ITEM_ID)p_menu_set->menu_item_count;
- Line 250 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 286 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  else
                                                  {
                                                          q = p;
-                                                         while (isalnum(*q) || *q == '_')
+                                                         while (isalnum((int)*q) || *q == '_' || *q == '-')
                                                          {
                                                                  q++;
                                                          }
                                                          if (*q != '\0')
                                                          {
-                                                                 log_error("Error menu item action in menu config line %d\n", fin_line);
+                                                                 log_error("Error menu item action in menu config line %d", fin_line);
                                                                  return -1;
                                                          }
                                                  }
                                                  if (q - p > sizeof(p_menu_item->action) - 1)
                                                  {
-                                                         log_error("Too longer menu action in menu config line %d\n", fin_line);
+                                                         log_error("Too longer menu action in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  strncpy(p_menu_item->action, p, sizeof(p_menu_item->action) - 1);
- Line 273 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 309 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu item row in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu item row in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu_item->row = (int16_t)atoi(p);
- Line 292 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 328 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu item col in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item col in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu item col in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item col in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu_item->col = (int16_t)atoi(p);
- Line 311 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 347 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu item priv in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item priv in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu item priv in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item priv in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu_item->priv = atoi(p);
- Line 330 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 366 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu item level in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item level in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu item level in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item level in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu_item->level = atoi(p);
- Line 349 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 385 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL || *q != '\"')
                                                  {
-                                                         log_error("Error menu item name in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item name in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  q++;
                                                  p = q;
                                                  while (*q != '\0' && *q != '\"')
                                                  {
+                                                         if (*q == '\\')
+                                                         {
+                                                                 r = q;
+                                                                 while (*r != '\0')
+                                                                 {
+                                                                         *r = *(r + 1);
+                                                                         r++;
+                                                                 }
+                                                         }
                                                          q++;
                                                  }
                                                  if (*q != '\"' || *(q + 1) != '\0')
                                                  {
-                                                         log_error("Error menu item name in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item name in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  *q = '\0';
                                                  if (q - p > sizeof(p_menu_item->name) - 1)
                                                  {
-                                                         log_error("Too longer menu name in menu config line %d\n", fin_line);
+                                                         log_error("Too longer menu name in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  strncpy(p_menu_item->name, p, sizeof(p_menu_item->name) - 1);
- Line 377 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 422 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITHOUT_SPACE, &saveptr);
                                                  if (q == NULL || (q = strchr(q, '\"')) == NULL)
                                                  {
-                                                         log_error("Error menu item text in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item text in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  q++;
                                                  p = q;
                                                  while (*q != '\0' && *q != '\"')
                                                  {
+                                                         if (*q == '\\')
+                                                         {
+                                                                 r = q;
+                                                                 while (*r != '\0')
+                                                                 {
+                                                                         *r = *(r + 1);
+                                                                         r++;
+                                                                 }
+                                                         }
                                                          q++;
                                                  }
                                                  if (*q != '\"')
                                                  {
-                                                         log_error("Error menu item text in menu config line %d\n", fin_line);
+                                                         log_error("Error menu item text in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  *q = '\0';
                                                  if (q - p > sizeof(p_menu_item->text) - 1)
                                                  {
-                                                         log_error("Too longer menu item text in menu config line %d\n", fin_line);
+                                                         log_error("Too longer menu item text in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  strncpy(p_menu_item->text, p, sizeof(p_menu_item->text) - 1);
- Line 405 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 459 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(q + 1, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q != NULL)
                                                  {
-                                                         log_error("Unknown extra content in menu config line %d\n", fin_line);
+                                                         log_error("Unknown extra content in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                          }
- Line 417 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 471 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu title row in menu config line %d\n", fin_line);
+                                                         log_error("Error menu title row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu title row in menu config line %d\n", fin_line);
+                                                         log_error("Error menu title row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu->title.row = (int16_t)atoi(p);
- Line 436 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 490 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu title col in menu config line %d\n", fin_line);
+                                                         log_error("Error menu title col in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu title col in menu config line %d\n", fin_line);
+                                                         log_error("Error menu title col in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu->title.col = (int16_t)atoi(p);
- Line 455 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 509 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITHOUT_SPACE, &saveptr);
                                                  if (q == NULL || (q = strchr(q, '\"')) == NULL)
                                                  {
-                                                         log_error("Error menu title text in menu config line %d\n", fin_line);
+                                                         log_error("Error menu title text in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  q++;
                                                  p = q;
                                                  while (*q != '\0' && *q != '\"')
                                                  {
+                                                         if (*q == '\\')
+                                                         {
+                                                                 r = q;
+                                                                 while (*r != '\0')
+                                                                 {
+                                                                         *r = *(r + 1);
+                                                                         r++;
+                                                                 }
+                                                         }
                                                          q++;
                                                  }
                                                  if (*q != '\"')
                                                  {
-                                                         log_error("Error menu title text in menu config line %d\n", fin_line);
+                                                         log_error("Error menu title text in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  *q = '\0';
-                                                 if (q - p > sizeof(p_menu_item->text) - 1)
+                                                 if (q - p > sizeof(p_menu->title.text) - 1)
                                                  {
-                                                         log_error("Too longer menu title text in menu config line %d\n", fin_line);
+                                                         log_error("Too longer menu title text in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  strncpy(p_menu->title.text, p, sizeof(p_menu->title.text) - 1);
- Line 483 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 546 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(q + 1, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q != NULL)
                                                  {
-                                                         log_error("Unknown extra content in menu config line %d\n", fin_line);
+                                                         log_error("Unknown extra content in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                          }
- Line 495 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 558 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu screen row in menu config line %d\n", fin_line);
+                                                         log_error("Error menu screen row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu screen row in menu config line %d\n", fin_line);
+                                                         log_error("Error menu screen row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu->screen_row = (int16_t)atoi(p);
- Line 514 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 577 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu screen col in menu config line %d\n", fin_line);
+                                                         log_error("Error menu screen col in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isdigit(*q))
+                                                 while (isdigit((int)*q))
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu screen col in menu config line %d\n", fin_line);
+                                                         log_error("Error menu screen col in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p_menu->screen_col = (int16_t)atoi(p);
- Line 533 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 596 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q == NULL)
                                                  {
-                                                         log_error("Error menu screen name in menu config line %d\n", fin_line);
+                                                         log_error("Error menu screen name in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                                  p = q;
-                                                 while (isalnum(*q) || *q == '_')
+                                                 while (isalnum((int)*q) || *q == '_' || *q == '-')
                                                  {
                                                          q++;
                                                  }
                                                  if (*q != '\0')
                                                  {
-                                                         log_error("Error menu screen name in menu config line %d\n", fin_line);
+                                                         log_error("Error menu screen name in menu config line %d", fin_line);
+                                                         return -1;
+                                                 }
+                                                 strncpy(p_menu->screen_name, p, sizeof(p_menu->screen_name) - 1);
+                                                 p_menu->screen_name[sizeof(p_menu->screen_name) - 1] = '\0';
+                                                 // Check syntax
+                                                 q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
+                                                 if (q != NULL)
+                                                 {
+                                                         log_error("Unknown extra content in menu config line %d", fin_line);
+                                                         return -1;
+                                                 }
+                                         }
+                                         else if (strcmp(p, "page") == 0)
+                                         {
+                                                 // Menu page row
+                                                 q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
+                                                 if (q == NULL)
+                                                 {
+                                                         log_error("Error menu page row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
-                                                 if (trie_dict_get(p_menu_set->p_menu_screen_dict, p, (int64_t *)&screen_id) != 1)
+                                                 p = q;
+                                                 while (isdigit((int)*q))
+                                                 {
+                                                         q++;
+                                                 }
+                                                 if (*q != '\0')
                                                  {
-                                                         log_error("Undefined menu screen [%s]\n", p);
+                                                         log_error("Error menu page row in menu config line %d", fin_line);
                                                          return -1;
                                                  }
-                                                 p_menu->screen_id = screen_id;
+                                                 p_menu->page_row = (int16_t)atoi(p);
+                                                 // Menu page col
+                                                 q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
+                                                 if (q == NULL)
+                                                 {
+                                                         log_error("Error menu page col in menu config line %d", fin_line);
+                                                         return -1;
+                                                 }
+                                                 p = q;
+                                                 while (isdigit((int)*q))
+                                                 {
+                                                         q++;
+                                                 }
+                                                 if (*q != '\0')
+                                                 {
+                                                         log_error("Error menu page col in menu config line %d", fin_line);
+                                                         return -1;
+                                                 }
+                                                 p_menu->page_col = (int16_t)atoi(p);
+                                                 // Menu page item limit
+                                                 q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
+                                                 if (q == NULL)
+                                                 {
+                                                         log_error("Error menu page item limit in menu config line %d", fin_line);
+                                                         return -1;
+                                                 }
+                                                 p = q;
+                                                 while (isdigit((int)*q))
+                                                 {
+                                                         q++;
+                                                 }
+                                                 if (*q != '\0')
+                                                 {
+                                                         log_error("Error menu page item limit in menu config line %d", fin_line);
+                                                         return -1;
+                                                 }
+                                                 p_menu->page_item_limit = (int16_t)atoi(p);
+                                                 // Check syntax
+                                                 q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
+                                                 if (q != NULL)
+                                                 {
+                                                         log_error("Unknown extra content in menu config line %d", fin_line);
+                                                         return -1;
+                                                 }
+                                         }
+                                         else if (strcmp(p, "use_filter") == 0)
+                                         {
+                                                 p_menu->use_filter = 1;
                                                  // Check syntax
                                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                                  if (q != NULL)
                                                  {
-                                                         log_error("Unknown extra content in menu config line %d\n", fin_line);
+                                                         log_error("Unknown extra content in menu config line %d", fin_line);
                                                          return -1;
                                                  }
                                          }
- Line 567 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 705 
  int load_menu(MENU_SET *p_menu_set, cons
                          {
                                  if (p_menu_set->menu_item_count >= MAX_MENUS)
                                  {
-                                         log_error("Menu screen count (%d) exceed limit (%d)\n", p_menu_set->menu_screen_count, MAX_MENUS);
+                                         log_error("Menu screen count (%d) exceed limit (%d)", p_menu_set->menu_screen_count, MAX_MENUS);
                                          return -3;
                                  }
                                  screen_id = (MENU_SCREEN_ID)p_menu_set->menu_screen_count;
- Line 576 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 714 
  int load_menu(MENU_SET *p_menu_set, cons
                                  p_screen = get_menu_screen_by_id(p_menu_set, screen_id);
                                  q = p;
-                                 while (isalnum(*q) || *q == '_')
+                                 while (isalnum((int)*q) || *q == '_' || *q == '-')
                                  {
                                          q++;
                                  }
                                  if (*q != '\0')
                                  {
-                                         log_error("Error menu screen name in menu config line %d\n", fin_line);
+                                         log_error("Error menu screen name in menu config line %d", fin_line);
                                          return -1;
                                  }
                                  strncpy(p_screen->name, p, sizeof(p_screen->name) - 1);
- Line 590 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 728 
  int load_menu(MENU_SET *p_menu_set, cons
                                  if (trie_dict_set(p_menu_set->p_menu_screen_dict, p_screen->name, (int64_t)screen_id) != 1)
                                  {
-                                         log_error("Error set menu screen dict [%s]\n", p_screen->name);
+                                         log_error("Error set menu screen dict [%s]", p_screen->name);
                                  }
                                  // Check syntax
                                  q = strtok_r(NULL, MENU_CONF_DELIM_WITH_SPACE, &saveptr);
                                  if (q != NULL)
                                  {
-                                         log_error("Unknown extra content in menu config line %d\n", fin_line);
+                                         log_error("Unknown extra content in menu config line %d", fin_line);
                                          return -1;
                                  }
- Line 618 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 756 
  int load_menu(MENU_SET *p_menu_set, cons
                                          {
                                                  if (p_menu_set->p_menu_screen_buf_free + 1 > q)
                                                  {
-                                                         log_error("Menu screen buffer depleted (%p + 1 > %p)\n", p_menu_set->p_menu_screen_buf_free, q);
+                                                         log_error("Menu screen buffer depleted (%p + 1 > %p)", p_menu_set->p_menu_screen_buf_free, q);
                                                          return -3;
                                                  }
- Line 628 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 766 
  int load_menu(MENU_SET *p_menu_set, cons
                                                  break;
                                          }
+                                         // Clear line
+                                         if (p_menu_set->p_menu_screen_buf_free + strlen(CTRL_SEQ_CLR_LINE) > q)
+                                         {
+                                                 log_error("Menu screen buffer depleted (%p + %d > %p)", p_menu_set->p_menu_screen_buf_free, q, strlen(CTRL_SEQ_CLR_LINE));
+                                                 return -3;
+                                         }
+                                         p_menu_set->p_menu_screen_buf_free = stpcpy(p_menu_set->p_menu_screen_buf_free, CTRL_SEQ_CLR_LINE);
                                          p = buffer;
                                          while (*p != '\0')
                                          {
                                                  if (p_menu_set->p_menu_screen_buf_free + 2 > q)
                                                  {
-                                                         log_error("Menu screen buffer depleted (%p + 2 > %p)\n", p_menu_set->p_menu_screen_buf_free, q);
+                                                         log_error("Menu screen buffer depleted (%p + 2 > %p)", p_menu_set->p_menu_screen_buf_free, q);
                                                          return -3;
                                                  }
- Line 651 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 797 
  int load_menu(MENU_SET *p_menu_set, cons
                                  if (p_screen->buf_length == -1)
                                  {
-                                         log_error("End of menu screen [%s] not found\n", p_screen->name);
+                                         log_error("End of menu screen [%s] not found", p_screen->name);
                                  }
                          }
                  }
                  else // Invalid prefix
                  {
-                         log_error("Error in menu config line %d\n", fin_line);
+                         log_error("Error in menu config line %d", fin_line);
                          return -1;
                  }
          }
          fclose(fin);
-         // Set menu_item->action_menu_id of each menu item pointing to a submenu to the menu_id of the corresponding submenu
+         for (menu_id = 0; menu_id < p_menu_set->menu_count; menu_id++)
+         {
+                 p_menu = get_menu_by_id(p_menu_set, menu_id);
+                 if (trie_dict_get(p_menu_set->p_menu_screen_dict, p_menu->screen_name, (int64_t *)(&(p_menu->screen_id))) != 1)
+                 {
+                         log_error("Undefined menu screen [%s]", p);
+                         return -1;
+                 }
+                 // Set menu->filter_handler of each menu pointing to filter
+                 if (p_menu->use_filter == 1)
+                 {
+                         if ((p_menu->filter_handler = get_cmd_handler(p_menu->name)) == NULL)
+                         {
+                                 log_error("Undefined menu filter handler [%s]", p_menu->name);
+                                 return -1;
+                         }
+                 }
+         }
          for (menu_item_id = 0; menu_item_id < p_menu_set->menu_item_count; menu_item_id++)
          {
                  p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
-                 if (p_menu_item->submenu == 1 && strcmp(p_menu_item->action, "..") != 0)
+                 // Set menu_item->action_cmd_handler of each menu item pointing to bbs_cmd
+                 if (p_menu_item->submenu == 0)
+                 {
+                         if ((p_menu_item->action_cmd_handler = get_cmd_handler(p_menu_item->action)) == NULL)
+                         {
+                                 log_error("Undefined menu action cmd handler [%s]", p_menu_item->action);
+                                 return -1;
+                         }
+                 }
+                 // Set menu_item->action_menu_id of each menu item pointing to a submenu to the menu_id of the corresponding submenu
+                 else if (strcmp(p_menu_item->action, "..") != 0)
                  {
                          if (trie_dict_get(p_menu_set->p_menu_name_dict, p_menu_item->action, (int64_t *)&menu_id) != 1)
                          {
-                                 log_error("Undefined menu action [%s]\n", p_menu_item->action);
+                                 log_error("Undefined sub menu id [%s]", p_menu_item->action);
                                  return -1;
                          }
                          p_menu_item->action_menu_id = menu_id;
- Line 686 
  int load_menu(MENU_SET *p_menu_set, cons
+ Line 863 
  int load_menu(MENU_SET *p_menu_set, cons
          return 0;
  }
- static int display_menu_cursor(MENU_SET *p_menu_set, int show)
+ int display_menu_cursor(MENU_SET *p_menu_set, int show)
  {
          MENU_ID menu_id;
          MENU_ITEM_ID menu_item_id;
- Line 698 
  static int display_menu_cursor(MENU_SET
+ Line 875 
  static int display_menu_cursor(MENU_SET
          p_menu = get_menu_by_id(p_menu_set, menu_id);
          if (p_menu == NULL)
          {
-                 log_error("get_menu_by_id(%d) return NULL pointer\n", menu_id);
+                 log_error("get_menu_by_id(%d) return NULL pointer", menu_id);
                  return -1;
          }
- Line 707 
  static int display_menu_cursor(MENU_SET
+ Line 884 
  static int display_menu_cursor(MENU_SET
          p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
          if (p_menu_item == NULL)
          {
-                 log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                 log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
                  return -1;
          }
          moveto(p_menu_set->menu_item_r_row[menu_item_pos], p_menu_set->menu_item_r_col[menu_item_pos] - 2);
          outc(show ? '>' : ' ');
-         iflush();
          return 0;
  }
- int display_menu(MENU_SET *p_menu_set)
+ static int display_menu_current_page(MENU_SET *p_menu_set)
  {
          int16_t row = 0;
          int16_t col = 0;
-         int menu_selectable = 0;
          MENU_ID menu_id;
          MENU_ITEM_ID menu_item_id;
          MENU *p_menu;
          MENU_ITEM *p_menu_item;
-         MENU_SCREEN *p_menu_screen;
          int16_t menu_item_pos;
+         int16_t page_id = 0;
+         MENU_SCREEN *p_menu_screen;
          menu_id = p_menu_set->menu_id_path[p_menu_set->choose_step];
          p_menu = get_menu_by_id(p_menu_set, menu_id);
          if (p_menu == NULL)
          {
-                 log_error("get_menu_by_id(%d) return NULL pointer\n", menu_id);
+                 log_error("get_menu_by_id(%d) return NULL pointer", menu_id);
                  return -1;
          }
-         menu_item_pos = p_menu_set->menu_item_pos[p_menu_set->choose_step];
+         clrline(p_menu->page_row, p_menu->page_row + p_menu->page_item_limit - 1);
-         menu_item_id = p_menu->items[menu_item_pos];
-         p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
-         if (p_menu_item == NULL)
-         {
-                 log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
-                 return -1;
-         }
-         if (menu_item_pos > 0 &&
-                 checkpriv(&BBS_priv, 0, p_menu_item->priv) != 0 &&
-                 checklevel(&BBS_priv, p_menu_item->level) != 0)
-         {
-                 menu_selectable = 1;
-         }
          if (p_menu->title.show)
          {
-                 show_top(p_menu->title.text);
+                 if (p_menu->title.row == 0 && p_menu->title.col == 0)
+                 {
+                         show_top(p_menu->title.text, BBS_name, "");
+                 }
+                 else
+                 {
+                         moveto(p_menu->title.row, p_menu->title.col);
+                         prints("%s", p_menu->title.text);
+                 }
          }
          if (p_menu->screen_show)
- Line 764 
  int display_menu(MENU_SET *p_menu_set)
+ Line 934 
  int display_menu(MENU_SET *p_menu_set)
                  p_menu_screen = get_menu_screen_by_id(p_menu_set, p_menu->screen_id);
                  if (p_menu_screen == NULL)
                  {
-                         log_error("get_menu_screen_by_id(%d) return NULL pointer\n", p_menu->screen_id);
+                         log_error("get_menu_screen_by_id(%d) return NULL pointer", p_menu->screen_id);
                          return -1;
                  }
-                 moveto(p_menu->screen_row, p_menu->screen_col);
+                 row = p_menu->screen_row;
+                 col = p_menu->screen_col;
+                 moveto(row, col);
                  prints("%s", p_menu_set->p_menu_screen_buf + p_menu_screen->buf_offset);
-                 iflush();
+         }
+         menu_item_pos = p_menu_set->menu_item_pos[p_menu_set->choose_step];
+         page_id = p_menu_set->menu_item_page_id[menu_item_pos];
+         while (menu_item_pos >= 0)
+         {
+                 if (p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                 {
+                         menu_item_pos++;
+                         break;
+                 }
+                 if (menu_item_pos == 0)
+                 {
+                         break;
+                 }
+                 menu_item_pos--;
+         }
+         for (; menu_item_pos < p_menu->item_count; menu_item_pos++)
+         {
+                 if (p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                 {
+                         break;
+                 }
+                 menu_item_id = p_menu->items[menu_item_pos];
+                 p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
+                 if (p_menu_set->menu_item_display[menu_item_pos] == 0)
+                 {
+                         continue;
+                 }
+                 row = p_menu_set->menu_item_r_row[menu_item_pos];
+                 col = p_menu_set->menu_item_r_col[menu_item_pos];
+                 moveto(row, col);
+                 prints("%s", p_menu_item->text);
+         }
+         return 0;
+ }
+ int display_menu(MENU_SET *p_menu_set)
+ {
+         int16_t row = 0;
+         int16_t col = 0;
+         int menu_selectable = 0;
+         MENU_ID menu_id;
+         MENU_ITEM_ID menu_item_id;
+         MENU *p_menu;
+         MENU_ITEM *p_menu_item;
+         int16_t menu_item_pos;
+         int16_t page_id = 0;
+         int page_item_count = 0;
+         menu_id = p_menu_set->menu_id_path[p_menu_set->choose_step];
+         p_menu = get_menu_by_id(p_menu_set, menu_id);
+         if (p_menu == NULL)
+         {
+                 log_error("get_menu_by_id(%d) return NULL pointer", menu_id);
+                 if (p_menu_set->choose_step > 0)
+                 {
+                         p_menu_set->choose_step--;
+                         return REDRAW;
+                 }
+                 return EXITMENU;
+         }
+         if (p_menu->item_count <= 0) // empty menu
+         {
+                 moveto(p_menu->screen_row, p_menu->screen_col);
+                 clrtoeol();
+                 prints("没有可选项");
+                 press_any_key();
+                 return -1;
+         }
+         menu_item_pos = p_menu_set->menu_item_pos[p_menu_set->choose_step];
+         menu_item_id = p_menu->items[menu_item_pos];
+         p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
+         if (p_menu_item == NULL)
+         {
+                 log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
+                 return EXITMENU;
+         }
+         if (menu_item_pos > 0 &&
+                 !(p_menu->use_filter ? (p_menu->filter_handler((void *)p_menu_item) == 0)
+                                                          : (checkpriv(&BBS_priv, 0, p_menu_item->priv) == 0 ||
+                                                                 checklevel2(&BBS_priv, p_menu_item->level) == 0)))
+         {
+                 menu_selectable = 1;
          }
          for (menu_item_pos = 0; menu_item_pos < p_menu->item_count; menu_item_pos++)
- Line 787 
  int display_menu(MENU_SET *p_menu_set)
+ Line 1055 
  int display_menu(MENU_SET *p_menu_set)
                          col = p_menu_item->col;
                  }
-                 if (checkpriv(&BBS_priv, 0, p_menu_item->priv) == 0 || checklevel(&BBS_priv, p_menu_item->level) == 0)
+                 p_menu_set->menu_item_page_id[menu_item_pos] = page_id;
+                 if (p_menu->use_filter ? (p_menu->filter_handler((void *)p_menu_item) == 0)
+                                                            : (checkpriv(&BBS_priv, 0, p_menu_item->priv) == 0 ||
+                                                                   checklevel2(&BBS_priv, p_menu_item->level) == 0))
                  {
                          p_menu_set->menu_item_display[menu_item_pos] = 0;
                          p_menu_set->menu_item_r_row[menu_item_pos] = 0;
- Line 806 
  int display_menu(MENU_SET *p_menu_set)
+ Line 1078 
  int display_menu(MENU_SET *p_menu_set)
                          p_menu_set->menu_item_r_row[menu_item_pos] = row;
                          p_menu_set->menu_item_r_col[menu_item_pos] = col;
-                         moveto(row, col);
-                         prints("%s", p_menu_item->text);
                          row++;
+                         page_item_count++;
+                         if (p_menu->page_item_limit > 0 && page_item_count >= p_menu->page_item_limit)
+                         {
+                                 page_id++;
+                                 page_item_count = 0;
+                                 row = p_menu->page_row;
+                                 col = p_menu->page_col;
+                         }
                  }
          }
          if (!menu_selectable)
          {
+                 moveto(p_menu->screen_row, p_menu->screen_col);
+                 clrtoeol();
+                 prints("没有可选项");
+                 press_any_key();
+                 return -1;
+         }
+         if (display_menu_current_page(p_menu_set) != 0)
+         {
                  return -1;
          }
- Line 830 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1117 
  int menu_control(MENU_SET *p_menu_set, i
          MENU *p_menu;
          MENU_ITEM *p_menu_item;
          int16_t menu_item_pos;
+         int16_t page_id;
+         int require_page_change = 0;
          if (p_menu_set->menu_count == 0)
          {
-                 return 0;
+                 log_error("Empty menu set");
+                 return EXITBBS;
          }
          menu_id = p_menu_set->menu_id_path[p_menu_set->choose_step];
          p_menu = get_menu_by_id(p_menu_set, menu_id);
          if (p_menu == NULL)
          {
-                 log_error("get_menu_by_id(%d) return NULL pointer\n", menu_id);
+                 log_error("get_menu_by_id(%d) return NULL pointer", menu_id);
-                 return -1;
+                 if (p_menu_set->choose_step > 0)
+                 {
+                         p_menu_set->choose_step--;
+                         return REDRAW;
+                 }
+                 return EXITBBS;
          }
          if (p_menu->item_count == 0)
          {
-                 return 0;
+                 log_debug("Empty menu (%s)", p_menu->name);
+                 if (p_menu_set->choose_step > 0)
+                 {
+                         p_menu_set->choose_step--;
+                         return REDRAW;
+                 }
+                 return EXITBBS;
          }
          menu_item_pos = p_menu_set->menu_item_pos[p_menu_set->choose_step];
+         page_id = p_menu_set->menu_item_page_id[menu_item_pos];
          menu_item_id = p_menu->items[menu_item_pos];
          p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
          if (p_menu_item == NULL)
          {
-                 log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                 log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
-                 return -1;
+                 p_menu_set->menu_item_pos[p_menu_set->choose_step] = 0;
+                 return REDRAW;
          }
          switch (key)
          {
          case CR:
-                 igetch_reset();
          case KEY_RIGHT:
                  if (p_menu_item->submenu)
                  {
- Line 880 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1183 
  int menu_control(MENU_SET *p_menu_set, i
                  }
                  else
                  {
-                         return (exec_cmd(p_menu_item->action, p_menu_item->name));
+                         return ((*(p_menu_item->action_cmd_handler))((void *)(p_menu_item->name)));
                  }
                  break;
+         case KEY_ESC:
          case KEY_LEFT:
                  if (p_menu_set->choose_step > 0)
                  {
                          p_menu_set->choose_step--;
-                         if (display_menu(p_menu_set) != 0)
+                         return REDRAW;
-                         {
-                                 return menu_control(p_menu_set, KEY_LEFT);
-                         }
                  }
                  else
                  {
+                         if (p_menu_set->allow_exit)
+                         {
+                                 return EXITMENU;
+                         }
                          display_menu_cursor(p_menu_set, 0);
                          menu_item_pos = p_menu->item_count - 1;
                          while (menu_item_pos >= 0)
- Line 902 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1208 
  int menu_control(MENU_SET *p_menu_set, i
                                  p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
                                  if (p_menu_item == NULL)
                                  {
-                                         log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                                         log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
                                          return -1;
                                  }
- Line 919 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1225 
  int menu_control(MENU_SET *p_menu_set, i
                          display_menu_cursor(p_menu_set, 1);
                  }
                  break;
+         case KEY_PGUP:
+                 require_page_change = 1;
          case KEY_UP:
                  display_menu_cursor(p_menu_set, 0);
                  do
- Line 927 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1235 
  int menu_control(MENU_SET *p_menu_set, i
                          if (menu_item_pos < 0)
                          {
                                  menu_item_pos = p_menu->item_count - 1;
+                                 require_page_change = 0;
                          }
                          menu_item_id = p_menu->items[menu_item_pos];
                          p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
                          if (p_menu_item == NULL)
                          {
-                                 log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                                 log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
                                  return -1;
                          }
-                 } while (!p_menu_set->menu_item_display[menu_item_pos]);
+                         if (require_page_change && p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                         {
+                                 require_page_change = 0;
+                         }
+                 } while (require_page_change || !p_menu_set->menu_item_display[menu_item_pos]);
                  p_menu_set->menu_item_pos[p_menu_set->choose_step] = menu_item_pos;
+                 if (p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                 {
+                         display_menu_current_page(p_menu_set);
+                 }
                  display_menu_cursor(p_menu_set, 1);
                  break;
+         case KEY_PGDN:
+                 require_page_change = 1;
          case KEY_DOWN:
                  display_menu_cursor(p_menu_set, 0);
                  do
- Line 947 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1266 
  int menu_control(MENU_SET *p_menu_set, i
                          if (menu_item_pos >= p_menu->item_count)
                          {
                                  menu_item_pos = 0;
+                                 require_page_change = 0;
                          }
                          menu_item_id = p_menu->items[menu_item_pos];
                          p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
                          if (p_menu_item == NULL)
                          {
-                                 log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                                 log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
                                  return -1;
                          }
-                 } while (!p_menu_set->menu_item_display[menu_item_pos]);
+                         if (require_page_change && p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                         {
+                                 require_page_change = 0;
+                         }
+                 } while (require_page_change || !p_menu_set->menu_item_display[menu_item_pos]);
                  p_menu_set->menu_item_pos[p_menu_set->choose_step] = menu_item_pos;
+                 if (p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                 {
+                         display_menu_current_page(p_menu_set);
+                 }
                  display_menu_cursor(p_menu_set, 1);
                  break;
-         case KEY_PGUP:
+         case KEY_HOME:
                  display_menu_cursor(p_menu_set, 0);
                  menu_item_pos = 0;
                  while (menu_item_pos < p_menu->item_count - 1)
- Line 968 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1296 
  int menu_control(MENU_SET *p_menu_set, i
                          p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
                          if (p_menu_item == NULL)
                          {
-                                 log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                                 log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
                                  return -1;
                          }
- Line 980 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1308 
  int menu_control(MENU_SET *p_menu_set, i
                          menu_item_pos++;
                  }
                  p_menu_set->menu_item_pos[p_menu_set->choose_step] = menu_item_pos;
+                 if (p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                 {
+                         display_menu_current_page(p_menu_set);
+                 }
                  display_menu_cursor(p_menu_set, 1);
                  break;
-         case KEY_PGDN:
+         case KEY_END:
                  display_menu_cursor(p_menu_set, 0);
                  menu_item_pos = p_menu->item_count - 1;
                  while (menu_item_pos > 0)
- Line 991 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1323 
  int menu_control(MENU_SET *p_menu_set, i
                          p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
                          if (p_menu_item == NULL)
                          {
-                                 log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                                 log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
                                  return -1;
                          }
- Line 1003 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1335 
  int menu_control(MENU_SET *p_menu_set, i
                          menu_item_pos--;
                  }
                  p_menu_set->menu_item_pos[p_menu_set->choose_step] = menu_item_pos;
+                 if (p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                 {
+                         display_menu_current_page(p_menu_set);
+                 }
                  display_menu_cursor(p_menu_set, 1);
                  break;
          default:
- Line 1014 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1350 
  int menu_control(MENU_SET *p_menu_set, i
                                  p_menu_item = get_menu_item_by_id(p_menu_set, menu_item_id);
                                  if (p_menu_item == NULL)
                                  {
-                                         log_error("get_menu_item_by_id(%d) return NULL pointer\n", menu_item_id);
+                                         log_error("get_menu_item_by_id(%d) return NULL pointer", menu_item_id);
                                          return -1;
                                  }
- Line 1022 
  int menu_control(MENU_SET *p_menu_set, i
+ Line 1358 
  int menu_control(MENU_SET *p_menu_set, i
                                  {
                                          display_menu_cursor(p_menu_set, 0);
                                          p_menu_set->menu_item_pos[p_menu_set->choose_step] = menu_item_pos;
+                                         if (p_menu_set->menu_item_page_id[menu_item_pos] != page_id)
+                                         {
+                                                 display_menu_current_page(p_menu_set);
+                                         }
                                          display_menu_cursor(p_menu_set, 1);
-                                         return 0;
+                                         break;
                                  }
                          }
                  }
                  break;
          }
-         return 0;
+         return NOREDRAW;
  }
  int unload_menu(MENU_SET *p_menu_set)
  {
+         if (p_menu_set == NULL)
+         {
+                 log_error("NULL pointer error");
+                 return -1;
+         }
          if (p_menu_set->p_menu_name_dict != NULL)
          {
                  trie_dict_destroy(p_menu_set->p_menu_name_dict);
- Line 1047 
  int unload_menu(MENU_SET *p_menu_set)
+ Line 1393 
  int unload_menu(MENU_SET *p_menu_set)
                  p_menu_set->p_menu_screen_dict = NULL;
          }
-         unload_menu_shm(p_menu_set);
+         detach_menu_shm(p_menu_set);
-         if (shmctl(p_menu_set->shmid, IPC_RMID, NULL) == -1)
+         if (shm_unlink(p_menu_set->shm_name) == -1 && errno != ENOENT)
          {
-                 log_error("shmctl(shmid=%d, IPC_RMID) error (%d)\n", p_menu_set->shmid, errno);
+                 log_error("shm_unlink(%s) error (%d)", p_menu_set->shm_name, errno);
-                 return -1;
+                 return -2;
          }
          return 0;
  }
- int load_menu_shm(MENU_SET *p_menu_set)
+ int get_menu_shm_readonly(MENU_SET *p_menu_set)
  {
-         // Mount shared memory
+         int fd;
-         if (p_menu_set->p_reserved != NULL)
+         void *p_shm;
+         struct stat sb;
+         size_t size;
+         if (p_menu_set == NULL)
          {
-                 log_error("Menu is already loaded\n");
+                 log_error("NULL pointer error");
                  return -1;
          }
-         p_menu_set->p_reserved = shmat(p_menu_set->shmid, NULL, SHM_RDONLY);
+         if ((fd = shm_open(p_menu_set->shm_name, O_RDONLY, 0600)) == -1)
-         if (p_menu_set->p_reserved == (void *)-1)
          {
-                 log_error("shmat() error (%d)\n", errno);
+                 log_error("shm_open(%s) error (%d)", p_menu_set->shm_name, errno);
+                 return -2;
+         }
+         if (fstat(fd, &sb) < 0)
+         {
+                 log_error("fstat(fd) error (%d)", errno);
+                 close(fd);
+                 return -2;
+         }
+         size = (size_t)sb.st_size;
+         p_shm = mmap(NULL, size, PROT_READ, MAP_SHARED, fd, 0L);
+         if (p_shm == MAP_FAILED)
+         {
+                 log_error("mmap() error (%d)", errno);
+                 close(fd);
                  return -2;
          }
-         p_menu_set->p_menu_pool = p_menu_set->p_reserved + MENU_SET_RESERVED_LENGTH;
-         p_menu_set->p_menu_item_pool = p_menu_set->p_menu_pool + sizeof(MENU) * MAX_MENUS;
-         p_menu_set->p_menu_screen_pool = p_menu_set->p_menu_item_pool + sizeof(MENU_ITEM) * MAX_MENUITEMS;
-         p_menu_set->p_menu_screen_buf = p_menu_set->p_menu_screen_pool + sizeof(MENU_SCREEN) * MAX_MENUS;
-         p_menu_set->p_menu_screen_buf_free = p_menu_set->p_menu_screen_buf;
-         // Restore status varaibles into reserved memory area
+         if (close(fd) < 0)
-         p_menu_set->menu_count = *((int16_t *)p_menu_set->p_reserved);
+         {
-         p_menu_set->menu_item_count = *(((int16_t *)p_menu_set->p_reserved) + 1);
+                 log_error("close(fd) error (%d)", errno);
-         p_menu_set->menu_screen_count = *(((int16_t *)p_menu_set->p_reserved) + 2);
+                 return -1;
+         }
+         p_menu_set->shm_size = size;
+         p_menu_set->p_reserved = p_shm;
+         p_menu_set->p_menu_pool = (char *)(p_menu_set->p_reserved) + MENU_SET_RESERVED_LENGTH;
+         p_menu_set->p_menu_item_pool = (char *)(p_menu_set->p_menu_pool) + sizeof(MENU) * MAX_MENUS;
+         p_menu_set->p_menu_screen_pool = (char *)(p_menu_set->p_menu_item_pool) + sizeof(MENU_ITEM) * MAX_MENUITEMS;
+         p_menu_set->p_menu_screen_buf = (char *)(p_menu_set->p_menu_screen_pool) + sizeof(MENU_SCREEN) * MAX_MENUS;
+         p_menu_set->p_menu_screen_buf_free = p_menu_set->p_menu_screen_buf;
          p_menu_set->choose_step = 0;
          p_menu_set->menu_id_path[0] = 0;
+         p_menu_set->menu_item_pos[0] = 0;
-         p_menu_set->p_menu_name_dict = NULL;
+         return 0;
-         p_menu_set->p_menu_screen_dict = NULL;
+ }
+ int set_menu_shm_readonly(MENU_SET *p_menu_set)
+ {
+         if (p_menu_set == NULL)
+         {
+                 log_error("NULL pointer error");
+                 return -1;
+         }
+         if (p_menu_set->p_reserved != NULL && mprotect(p_menu_set->p_reserved, p_menu_set->shm_size, PROT_READ) < 0)
+         {
+                 log_error("mprotect() error (%d)", errno);
+                 return -2;
+         }
          return 0;
  }
- int unload_menu_shm(MENU_SET *p_menu_set)
+ int detach_menu_shm(MENU_SET *p_menu_set)
  {
+         if (p_menu_set == NULL)
+         {
+                 log_error("NULL pointer error");
+                 return -1;
+         }
          p_menu_set->menu_count = 0;
          p_menu_set->menu_item_count = 0;
          p_menu_set->menu_screen_count = 0;
- Line 1106 
  int unload_menu_shm(MENU_SET *p_menu_set
+ Line 1498 
  int unload_menu_shm(MENU_SET *p_menu_set
          p_menu_set->p_menu_screen_buf = NULL;
          p_menu_set->p_menu_screen_buf_free = NULL;
-         if (p_menu_set->p_reserved != NULL && shmdt(p_menu_set->p_reserved) == -1)
+         p_menu_set->p_menu_name_dict = NULL;
+         p_menu_set->p_menu_screen_dict = NULL;
+         if (p_menu_set->p_reserved != NULL && munmap(p_menu_set->p_reserved, p_menu_set->shm_size) < 0)
          {
-                 log_error("shmdt() error (%d)\n", errno);
+                 log_error("munmap() error (%d)", errno);
-                 return -1;
+                 return -2;
          }
          p_menu_set->p_reserved = NULL;
          return 0;

  Legend:

 
 
 Removed lines/characters
  
 
 
 Changed lines/characters
 
 
  
 Added lines/characters
 Legend:

 
 
 Removed lines/characters
  
 
 
 Changed lines/characters
 
 
  
 Added lines/characters
-Removed lines/characters
+Added lines/characters

webmaster@leafok.com	ViewVC Help
Powered by ViewVC 1.3.0-beta1