c程序上的分段错误

Question

我想做一个接收一些的 c 程序 arguments.The 第一个参数是一个文件，其他参数是 words.The 程序将为每个单词创建一个线程并计算出现的次数file.I 使用 shell 脚本 (prts.sh) 来计算 file.I 中单词的出现次数，但当我尝试运行时，我得到了分段错误。我正在通过结构发送文件，argument.i 认为问题是当我尝试访问我在线程中发送的结构的元素时 funciton.This 是我到目前为止编写的代码:

#include<pthread.h>
#include<stdlib.h>
#include<stdio.h>
#include<string.h>
#define N 1000
int sum=0;
int n;

typedef struct {
char arg1[N];
char arg2[N];
} Mesaj;

pthread_mutex_t m=PTHREAD_MUTEX_INITIALIZER;
void* func(void *p){
    Mesaj *msg_func=(Mesaj*)p;
    char cmd[N];
    char result[N];
    FILE *fp;
    pthread_mutex_lock(&m);
    sprintf(cmd,"/home/alexdamian/prts.sh %s %s",msg_func->arg1,msg_func->arg2);  
    fp = popen(cmd, "r");
    fgets(result, N, fp);
    pclose(fp); 
    int n=atoi((char*)result);
    sum=sum+n;
    pthread_mutex_unlock(&m);
    free(msg_func);
    return NULL;
}
int main(int argc,char *argv[]){

pthread_t *th=malloc(argc*sizeof(pthread_t));


int i;
for(i=1;i<argc;i++){

    Mesaj *msg=malloc(sizeof(Mesaj));
    strcpy(msg->arg1,argv[i]);
    strcpy(msg->arg2,argv[i+1]);    

    pthread_create(th+i,NULL,func,msg);
}

for(i=0;i<argc;i++){
    pthread_join(*(th+i),NULL);
}
printf("the sum is %d\n",sum);
free(th);
return 0;

}

Answer 1

您的代码需要一些修正：

样式和格式，代码一定要漂亮，好看。
不要在不需要时使用 malloc()。
可以使用索引表示法时，不要使用指针算术表示法。
始终检查函数的 return 值

实际问题：您遍历传递给程序的所有参数并访问最后一个参数之后的 1 个参数。

这个for循环：

int i;
for(i=1;i<argc;i++){

    Mesaj *msg=malloc(sizeof(Mesaj));
    strcpy(msg->arg1,argv[i]);
    strcpy(msg->arg2,argv[i+1]);    

    pthread_create(th+i,NULL,func,msg);
}

应该是：

int i;
for (i = 1 ; i < argc - 1 ; i++)
{    
    Mesaj *msg;

    msg = malloc(sizeof(*msg));
    if (msg == NULL) /* do anything except dereferencing `msg' */
        continue;
    strcpy(msg->arg1, argv[i]);
    strcpy(msg->arg2, argv[i + 1]);    

    pthread_create(&th[i], NULL, func, msg);
}

我修复了你的程序以展示一些你可以改进的地方：

#include <pthread.h>
#include <stdlib.h>
#include <stdio.h>
#include <string.h>

#define N 1000

typedef struct
{
    char arguments[2][N];
    int *data;
} Mesaj;
pthread_mutex_t mutex = PTHREAD_MUTEX_INITIALIZER;

void *
function(void *data)
{
    Mesaj      *message;
    int        *value;
    char        cmd[N];
    char        result[N];
    FILE       *pipe;
    const char *format;

    message = (Mesaj*) data;
    value   = (int *) message->data;
    format  = "/home/iharob/prts.sh %s %s";

    snprintf(cmd, sizeof(cmd), format, 
        message->arguments[0], message->arguments[1]);
    pipe = popen(cmd, "r");
    if (pipe != NULL)
    {
        fgets(result, sizeof(result), pipe);
        pclose(pipe);

        /* This section should be protected with mutex */
        pthread_mutex_lock(&mutex);
        *value = *value + strtol(result, NULL, 10);
        pthread_mutex_unlock(&mutex);
    }
    else
        fprintf(stderr, "cannot execute the command...\n");

    free(message);

    return NULL;
}

int main(int argc,char *argv[])
{
    pthread_t  thread[argc];
    int        i;
    int        value;

    value = 0;
    for (i = 1 ; i < argc - 1 ; i++)
    {
        Mesaj *message;

        message = malloc(sizeof(*message));
        if (message != NULL)
        {
            message->data = &value;

            strcpy(message->arguments[0], argv[i]);
            strcpy(message->arguments[1], argv[i + 1]);

            pthread_create(&thread[i], NULL, function, message);
        }
    }

    for (i = 1 ; i < argc - 1 ; i++)
        pthread_join(thread[i], NULL);
    printf("The sum is %d\n", value);

    return 0;
}

例如，您不需要在此程序中使用单个全局变量，您可以在堆栈中创建变量并将其作为消息的一部分传递，并且您只需要 malloc() 消息即可它在线程启动时存在，使用 VLAs¹ 你可以声明一个线程数组而不是 malloc()ing 一个，这没有错，但如果可以的话你应该避免它.

1_{Variable Length Arrays}

c程序上的分段错误

segmentation fault on c program

c

core

segmentation-fault