我正在尝试构建一个可以为许多客户服务的服务器。
服务器只做一个简单的工作:从客户端获取输入字符串,然后将每个字母更改为大写。
但问题是,当我尝试关闭一个客户端时,例如,键入“Ctrl-C”,然后我的操作系统将突然关闭。
我使用Ubuntu 10.10和CentOS来测试我的程序,但总是遇到同样的问题。 这是我的源代码:
/* client.c */
#include <stdio.h>
#include <signal.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <sys/socket.h>
#include <netinet/in.h>
#define MAXLINE 80
#define SERV_PORT 8000
int main (void)
{
struct sockaddr_in servaddr, cliaddr;
char buf[MAXLINE];
int sockfd, n;
sockfd = socket(AF_INET, SOCK_STREAM, 0);
bzero(&servaddr, sizeof(servaddr));
servaddr.sin_family = AF_INET;
inet_pton(AF_INET, "127.0.0.1", &servaddr.sin_addr);
servaddr.sin_port = htons(SERV_PORT);
connect(sockfd, (struct sockaddr *)&servaddr, sizeof(servaddr));
while (fgets(buf, MAXLINE, stdin) != NULL) {
write(sockfd, buf, strlen(buf));
n = read(sockfd, buf, MAXLINE);
if(n == 0) printf("Connect closed\n");
else write(STDOUT_FILENO, buf, n);
}
close(sockfd);
return 0;
}
/* server */
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <sys/socket.h>
#include <netinet/in.h>
#define MAXLINE 80
#define SERV_PORT 8000
void sigchld_func (int signo) {
wait(NULL);
}
int main (void)
{
struct sockaddr_in servaddr, cliaddr;
char buf[MAXLINE];
char str[INET_ADDRSTRLEN];
int listenfd, connfd;
socklen_t cliaddr_len;
int n, i;
pid_t pid;
signal(SIGCHLD, sigchld_func);
listenfd = socket(AF_INET, SOCK_STREAM, 0);
bzero(&servaddr, sizeof(servaddr));
servaddr.sin_family = AF_INET;
servaddr.sin_addr.s_addr = htonl(INADDR_ANY);
servaddr.sin_port = htons(SERV_PORT);
bind(listenfd, (struct sockaddr *)&servaddr, sizeof(servaddr));
listen(listenfd, 20);
printf("Accepting connections...\n");
while (1) {
cliaddr_len = sizeof(cliaddr);
connfd = accept(listenfd, (struct sockaddr *)&cliaddr, &cliaddr_len);
if ((pid = fork()) < 0) {
perror("fork error!");
exit(1);
} else if (pid > 0) {
close(connfd);
} else {
close(listenfd);
while (1) {
n = read(connfd, buf, MAXLINE);
if (n <= 0) {
printf("Connection closed\n");
break;
}
inet_ntop(AF_INET, &cliaddr.sin_addr, str, sizeof(str));
printf("Received from %s at port %d\n", str, ntohs(cliaddr.sin_port));
for (i = 0; i < n; i++)
buf[i] = toupper(buf[i]);
write(connfd, buf, n);
}
close(connfd);
}
}
return 0;
}
答案 0 :(得分:6)
你的服务器代码导致了一个fork炸弹,你没有正确看到它,因为你没有检查accept
的返回值。
根本原因是在读取/写入客户端的子节点中,在连接关闭后,您既不是在调用exit
,也不是从主节点返回。
因此子进程保持在顶级while(1)
循环中,尝试accept
listenfd
(但你关闭了那个,这很好)。 accept
失败,你无论如何都会分叉。紧密的循环循环会降低计算机的速度,调度程序无法正确处理它们(除非你有适当的反制措施)。
因此在close(connfd)
之后退出程序,并在代码中添加更多错误检查。