--- lbbs/src/net_server.c 2025/05/12 03:59:46 1.26 +++ lbbs/src/net_server.c 2025/05/13 07:30:53 1.30 @@ -36,8 +36,6 @@ #include #include -#define MAX_EVENTS 10 - int net_server(const char *hostaddr, in_port_t port) { unsigned int namelen; @@ -46,8 +44,6 @@ int net_server(const char *hostaddr, in_ struct sockaddr_in sin; struct epoll_event ev, events[MAX_EVENTS]; int nfds, epollfd; - sigset_t nsigset; - sigset_t osigset; siginfo_t siginfo; socket_server = socket(AF_INET, SOCK_STREAM, IPPROTO_TCP); @@ -55,7 +51,7 @@ int net_server(const char *hostaddr, in_ if (socket_server < 0) { log_error("Create socket failed\n"); - exit(1); + return -1; } sin.sin_family = AF_INET; @@ -77,13 +73,13 @@ int net_server(const char *hostaddr, in_ { log_error("Bind address %s:%u failed (%d)\n", inet_ntoa(sin.sin_addr), ntohs(sin.sin_port), errno); - exit(2); + return -1; } if (listen(socket_server, 10) < 0) { log_error("Socket listen failed (%d)\n", errno); - exit(3); + return -1; } strncpy(hostaddr_server, inet_ntoa(sin.sin_addr), sizeof(hostaddr_server) - 1); @@ -94,11 +90,6 @@ int net_server(const char *hostaddr, in_ log_std("Listening at %s:%d\n", hostaddr_server, port_server); - sigemptyset(&nsigset); - sigaddset(&nsigset, SIGHUP); - sigaddset(&nsigset, SIGCHLD); - sigaddset(&nsigset, SIGTERM); - epollfd = epoll_create1(0); if (epollfd < 0) { @@ -106,11 +97,15 @@ int net_server(const char *hostaddr, in_ return -1; } - ev.events = EPOLLIN | EPOLLET; + ev.events = EPOLLIN; ev.data.fd = socket_server; if (epoll_ctl(epollfd, EPOLL_CTL_ADD, socket_server, &ev) == -1) { log_error("epoll_ctl(socket_server) error (%d)\n", errno); + if (close(epollfd) < 0) + { + log_error("close(epoll) error (%d)\n"); + } return -1; } @@ -119,20 +114,21 @@ int net_server(const char *hostaddr, in_ while (!SYS_server_exit || SYS_child_process_count > 0) { - sigprocmask(SIG_BLOCK, &nsigset, &osigset); - while ((SYS_child_exit || SYS_server_exit) && SYS_child_process_count > 0) { + SYS_child_exit = 0; + siginfo.si_pid = 0; ret = waitid(P_ALL, 0, &siginfo, WEXITED | WNOHANG); if (ret == 0 && siginfo.si_pid > 0) { + SYS_child_exit = 1; // Retry waitid + SYS_child_process_count--; log_std("Child process (%d) exited\n", siginfo.si_pid); } else if (ret == 0) { - SYS_child_exit = 0; break; } else if (ret < 0) @@ -153,6 +149,8 @@ int net_server(const char *hostaddr, in_ if (SYS_menu_reload && !SYS_server_exit) { + SYS_menu_reload = 0; + if (reload_menu(&bbs_menu) < 0) { log_error("Reload menu failed\n"); @@ -161,11 +159,8 @@ int net_server(const char *hostaddr, in_ { log_std("Reload menu successfully\n"); } - SYS_menu_reload = 0; } - sigprocmask(SIG_SETMASK, &osigset, NULL); - nfds = epoll_wait(epollfd, events, MAX_EVENTS, 100); // 0.1 second if (nfds < 0) @@ -188,16 +183,24 @@ int net_server(const char *hostaddr, in_ { if (events[i].data.fd == socket_server) { - while (1) // Accept all incoming connections until error + while (!SYS_server_exit) // Accept all incoming connections until error { socket_client = accept(socket_server, (struct sockaddr *)&sin, &namelen); if (socket_client < 0) { - if (errno != EAGAIN && errno != EWOULDBLOCK && errno != ECONNABORTED && errno != EINTR) + if (errno == EAGAIN || errno == EWOULDBLOCK) + { + break; + } + else if (errno == EINTR) + { + continue; + } + else { log_error("accept(socket_server) error (%d)\n", errno); + break; } - break; } strncpy(hostaddr_client, inet_ntoa(sin.sin_addr), sizeof(hostaddr_client) - 1); @@ -221,6 +224,11 @@ int net_server(const char *hostaddr, in_ } } + if (close(epollfd) < 0) + { + log_error("close(epoll) error (%d)\n"); + } + fcntl(socket_server, F_SETFL, flags); if (close(socket_server) == -1)