接受系统调用中的分段错误

时间:2016-03-31 20:14:21

标签: c sockets pthreads posix ansi

我有以下代码作为接受传入套接字连接的服务器的主循环。

目前宏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实现。如何修复分段错误错误?

1 个答案:

答案 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;
}