C言語についてです(スレッド)

Question

スレッドについてなのですが、以下のコードでsegmentation faultが起きてしまいます・・
ミューテックスがうまくいってないのでしょうか？

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

struct stack {
    int buf[128];
    int count;
};
pthread_mutex_t mtx;

void stack_init(struct stack* stk)
{
    stk->count = 0;
}

void stack_push(struct stack* stk, int val)
{
    stk->buf[stk->count++] = val;
}

int stack_pop(struct stack* stk)
{
    if (stk->count <= 0) {
        fprintf(stderr, "ERROR: stack is empty\n");
        abort();
    }

    return stk->buf[--stk->count];
}

void* thread_entry(void* stk){
    pthread_mutex_lock(&mtx);
    for (int i=0; i < 2; i++){
        
       stack_push(stk,i);
       stack_pop(stk);
    
    }
    pthread_mutex_unlock(&mtx); 
    return NULL;

}


int main(int argc, char* argv[])
{
    struct stack stk;
    stack_init(&stk);
    int er,i;
    void* ret;
    pthread_t tid[1];


    er = pthread_mutex_init(&mtx,NULL);
    if (er != 0) goto error;

    for(i=0;i<2;i++){
    er = pthread_create(&tid[i], NULL, thread_entry, (void *)&stk);
    if (er != 0) goto error;
    }
    
    for(i=0;i<2;i++){
    pthread_join(tid[i], &ret);
    }

   

    return 0;

    error:
        fprintf(stderr,"ERROR\n");
        return -1;

   }

Comments

[nob.]

OS 処理系のバージョンなど、環境を書いてください。 私の環境（Ubuntu16.04.10, gcc 5.4.0)では faultしません。

[Cchan]

windows for ubuntu (bash)になります
gcc 5.5.0

Answer 1

mainの中の
pthread_t tid[1];
を
pthread_t tid[2];
に変えてください。
それでとりあえず、Segmentation faultは解決します。

Comments

[Cchan]

ありがとうございます！

Answer 2

直接の回答ではありません。ちょっと見ですが・・・

void* thread_entry(void* stk){
    pthread_mutex_lock(&mtx);
    for (int i=0; i < 2; i++){

       stack_push(stk,i);
       stack_pop(stk);

    }
    pthread_mutex_unlock(&mtx); 
    return NULL;

}

ですが、pushとpopを一緒にロックしてしまうのはおかしいかと？・・・これでは、同一スレッドのpush,popの動作が連続して起こってしまうので、複数のスレッドを使う意味が無いような・・・
・・・通常は、その操作（実際にpushあるいはpopする動作：stk->buf[stk->count++] = val;とか）にのみロックを掛けます。

Comments

[cateye]

通常、情報を入れるスレッドと取り出すスレッドは別物です。そのため上記のような書き方になりましたが、違う意図であればご容赦下さい。ちなみに、当方の環境（Linux-mint+clang8.0）ではtatsu99さんの指摘を修正した状態でエラーにはなっていません。$?=0

[Cchan]

有難うございます！