0

4番目の引数をpthread_create（）に渡す方法に関して奇妙な問題に遭遇しました。

もともと、私は次のようにコードを書きました：

auditLogEntry *newEntry = NULL;

// malloc and init the memory for newEntry
rc = audit_init_log_entry(&newEntry);
// wrapper of 'goto cleanup'
ERR_IF( rc != 0 );
...
rc2 = pthread_attr_init(&attr);
ERR_IF( rc2 != 0 );
rc2 = pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_DETACHED);
ERR_IF( rc2 != 0 );
rc2 = pthread_create(&syslog_thread, &attr, syslog_thread_handler, (void *)newEntry);
ERR_IF( rc2 != 0 );
newEntry = NULL;
...
cleanup:
pthread_attr_destroy(&attr);
if (newEntry != NULL)
{
    audit_free_log_entry(newEntry);
    newEntry = NULL;
}

static void *syslog_thread_handler(void *t)
{
    auditLogEntry *entry = (auditLogEntry *)t;
    ... // code using entry
    cleanup:
    audit_free_log_entry(entry);

    pthread_exit(0);
}

すべてが正常に動作します。

次に、次のように変更しました。

rc2 = pthread_create(&syslog_thread, &attr, syslog_thread_handler, (void *)&newEntry);
ERR_IF( rc2 != 0 );
...
cleanup:
pthread_attr_destroy(&attr);
if (rc != 0 && newEntry != NULL)
{
    audit_free_log_entry(newEntry);
    newEntry = NULL;
}

static void *syslog_thread_handler(void *t)
{
    auditLogEntry **entry = (auditLogEntry **)t;
    ... // code using *entry
    cleanup:
    audit_free_log_entry(*entry);
    *entry = NULL;

    pthread_exit(0);
}

上記の変更後、スレッドハンドラーは*entryを使用してログエントリデータにアクセスします。しかし、それはうまくいきませんでした。さらに悪いことに、プロセスコアがダンプされました。

'man pthread_create'を試しましたが、最後の引数をどのように渡すかについての特別な言及は見つかりませんでした。

ここで私の不正行為はありますか？

4

2 に答える 2

2

完全なコードを表示していないため、何が起こっているのかを判断するのは困難です。

ただし、&newEntryスタック上の変数へのポインタを提供します。関数newEntryが終了したなどの理由でスコープ外になった場合、他のスレッドには無効なポインターがあります。これは、スタック上の現在なくなっている場所を指しています。そして、そのようなポインタを逆参照すると、未定義の動作が発生します。

int *foo(void) 
{
   int x = 2;
   return &x;
}

void bar(void)
{
   int *x = foo();
   printf("%d\n", *x); //can't do this, x points to something
                       //on the stack in the foo function, 
                       //which isn't valid any more
}

foo（）関数が&x作成したスレッドに渡された場合も、同じ問題が発生します。

void foo(void) 
{
   int x = 2;
  ...
  pthread_create(&tid, bar, NULL, &x);
}

void * bar（void * arg）{int * x = arg;

   printf("%d\n", *x); //same problem here, x points to something
                       //on the stack in the foo function, which isn't valid 
                       //if the foo function ends. This concept is exactly the same
                       //if x had been a pointer inside the foo() function.
}

これがおそらくあなたのシナリオです：

int globalx = 2; //global variable
void foo(void) 
{
  int *x = malloc(sizeof(int));
  ...

  pthread_create(&tid, bar, NULL, &x); //we're still taking the address of 
                                       //a local variable. 
}

void *bar(void *arg) 
{
   int **x = arg;
   printf("%p\n", *x); //still the same problem here, x points to something
                       //on the stack in the foo function, which isn't valid 
}
于 2012-08-08T21:48:15.790 に答える
0

コアダンプが何からのものであっても、それはpthreadからのものではありません。最後の議論であなたがしていることは絶対に大丈夫です。

于 2012-08-08T21:43:00.593 に答える