c error in array segmentation fault

Question

I tried to run this code in terminal in osx and linux ubuntu :

#include <pthread.h>
#include <stdio.h>
#include <unistd.h>
int fact=1; //this data is shared by thread(s)
int n;
int x;
int *arrayName;

int main(int argc, char *argv[])
{

    if (argc != 3){ //check number of arguments
        printf("Must use: a.out <integer value>\n");
        return -1;
    }
    int x = atoi(argv[1]);
    n = atoi(argv[2]);
    if (n < 0){ //check second passed argument
        printf("%d Must be >=0\n", atoi(argv[2]));
        return -1;
    }
   arrayName = (int *) malloc(n * sizeof(int));
    pthread_t tid[n];

    for(int i=0;i<n;i++){
        pthread_create(&tid[i], NULL, (void *) i, NULL);
    }
    int i=0;
    while(i<n){
        pthread_join(tid[i],NULL);
        i++;
    }
    i=0;
    while (i<n) {
        printf("Thread is %d",arrayName[i]);
        i++;
    }
}
void *calculateMult(void *i) {
    int j = (int) i;
    arrayName[j] = x * j;
    return NULL;
};

I ran these commands in terminal :

cc -pthread main.c

./a.out 1 1

But it gives me segment fault : 11 in osx and segment fault (core dumped) in linux , WHY ??

if (argc != 3) ehhh? for a.out <integer value> ??

Sourav Ghosh
– Sourav Ghosh

2016-11-28 08:17:36 +00:00
Commented Nov 28, 2016 at 8:17 — Sourav Ghosh
– Sourav Ghosh, Commented Nov 28, 2016 at 8:17
to check if the number of argument more than 3 .

Jasem Al-Yafie
– Jasem Al-Yafie

2016-11-28 08:18:56 +00:00
Commented Nov 28, 2016 at 8:18 — Jasem Al-Yafie
– Jasem Al-Yafie, Commented Nov 28, 2016 at 8:18

Mohan · Accepted Answer · 2016-11-28 10:44:36Z

1

I think you need to change pthread_create call because you have passed the wrong argument in pthread_create. Also check return from pthread_create.

You need to something like this

int s = pthread_create(&tid[i], NULL, (void *)calculateMult,  (void *)&i);
if (s != 0)
       printf("pthread_create failed");

Also you need to change your function as:

void *calculateMult(void *i) {
    int *j = (int*) i;
    arrayName[*j] = x * (*j);
    return NULL;
};

so you are done.

edited Nov 28, 2016 at 10:44

answered Nov 28, 2016 at 10:09

Mohan

1,91122 silver badges39 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Sourav Ghosh · Accepted Answer · 2016-11-28 08:20:16Z

0

In your code, you're calling

 pthread_create(&tid[i], NULL, (void *) i, NULL);

where, the third argument i is an int but the expected argument is of type void *(*start_routine) (void *). This invokes undefined behavior.

You need to supply a function pointer, something like calculateMult or alike.

answered Nov 28, 2016 at 8:20

Sourav Ghosh

135k17 gold badges192 silver badges271 bronze badges

Collectives™ on Stack Overflow

c error in array segmentation fault

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related