我有以下代码作为接受传入套接字连接的服务器的主循环。
目前宏OperationMode定义为1,因此它将执行pthread逻辑。
for (hit = 1 ;; hit++) {
printf("Got here\n\n");
length = sizeof(cli_addr);
/* block waiting for clients */
socketfd = accept(listenfd, (struct sockaddr *) &cli_addr, &length);
if (socketfd < 0)
printf("ERROR system call - accept error\n");
else
{
printf("Testing\n\n\n");
#ifdef OperationMode
pthread_t thread_id;
if(pthread_create(&thread_id, NULL, attendFTP(socketfd, hit), NULL))
{
perror("could not create thread");
return 1;
}
#else
pid = fork();
if(pid==0)
{
ftp(socketfd, hit);
}
else
{
close(socketfd);
kill(pid, SIGCHLD);
}
#endif
}
}
我能够为第一个传入的套接字连接创建一个线程,但是一旦我遍历循环,就会在行中出现分段错误错误
socketfd = accept(listened, (struct sockaddr *) &cli_addr, &length);
我的attendFTP功能有以下代码
void *attendFTP(int fd, int hit)
{
ftp(fd, hit);
return NULL;
}
这适用于fork实现。如何修复分段错误错误?
答案 0 :(得分:1)
pthread_create(&thread_id, NULL, attendFTP(socketfd, hit), NULL);
此代码通过调用attendFTP()
给定参数的结果 - 并且此结果始终为NULL。
所以pthread_create试图在NULL地址启动一个函数,相应地失败了。
如果使用-pedantic
参数运行编译器,编译器会告诉您正在执行的操作是错误的。如果没有-pedantic
,gcc会允许某些“扩展程序”,这可能会隐藏错误。顺便说一句,这就是为什么-pedantic
在我看来是必须的。
你真正想要的是将一些参数传递给你的线程函数。不幸的是,它在C pthreads中确实令人费解,并且要求您分配和释放所述struct
。像这样:
struct args {
int fd;
int hit;
};
...
pthread_t thread_id;
struct args* args = malloc(sizeof(struct args));
args->fd = socketfd;
args->hit = hit;
if(pthread_create(&thread_id, NULL, attendFTP, args))
....
void* attendFTP(void* vargs)
{
struct args* args = vargs;
ftp(args->fd, args->hit);
free(args);
return NULL;
}