0

私は BSPlib を使用しており、複数のスレッドで実行されている関数に "int i" の単純な定義を追加すると (他の多くのスレッドと共に)、「プロセス 2 が SIGNAL 11 Segmantation fault をキャッチしました」のようなメッセージが表示されます。私は多くのことをチェックしましたが、それがなければセグメンテーション違反は発生せず、それを使用するとほとんど常に発生することに注意することが重要です。int 定義がどのようにそれを引き起こす可能性がありますか? 私が引き起こした可能性のあるスタックオーバーフローはありますか? ありがとう。

int P;
int main( int argc, char* argv[] )
{
    /** sequentail - process 0 */
    P=bsp_nprocs(); /// maximum number of process avialble (must do that on sequential part ,need for bsp begin)
    bsp_begin(P);
    char* str1;
    char* str2;
    int n;
    int** table;
    int thread=bsp_pid();
    int num_threads=bsp_nprocs();
    if(thread == 0)
    {
        ifstream file1(argv[1]);
        ifstream file2(argv[2]);
        // check if the strings are the same size RDBG
        string string1((istreambuf_iterator<char>(file1)), istreambuf_iterator<char>());
        string string2((istreambuf_iterator<char>(file2)), istreambuf_iterator<char>());
        n=string1.length();
        str1= (char*)malloc(sizeof(char)*(n+1));
        str2= (char*)malloc(sizeof(char)*(n+1));
        strcpy(str1,string1.c_str());
        strcpy(str2,string2.c_str());
    }
    if (thread!=0)
    {
        str1= (char*)malloc(sizeof(char)*(n+1));
        str2= (char*)malloc(sizeof(char)*(n+1));
    }
    bsp_push_reg(&n,SZINT);
    bsp_sync();
    bsp_get(0,&n,0,&n,SZINT);
    bsp_sync();
    if (thread==0)
    {
        table=(int**)malloc(sizeof(int)*(n+1));
        for (int i=0; i<n+1; i++)
            table[i]=(int*)malloc(sizeof(int)*(n+1));
    }
    bsp_push_reg(str1,SZCHAR*(n+1));
    bsp_push_reg(str2,SZCHAR*(n+1));
    bsp_push_reg(table,n*n*SZINT);
    bsp_sync();
    if (thread==0)
    {
        for(int t=1; t<num_threads; t++)
            for (int k=0; k<=n; k++)
            {
                bsp_put(t,str1+k,str1,k*SZCHAR,SZCHAR);
                bsp_put(t,str2+k,str2,k*SZCHAR,SZCHAR);
            }
    }
    bsp_sync();
    cout << thread << "!!!" << str1 << ";" << str2 << endl;
    int i;
    bsp_sync();
    bsp_pop_reg(table);
    bsp_pop_reg(str2);
    bsp_pop_reg(str1);
    bsp_pop_reg(&n);
    bsp_sync();
    free(str1);
    free(str2);
    bsp_sync();
    bsp_end();
    return 0;
}
4

3 に答える 3

2

テーブル変数の宣言/初期化が正しくありません。配列の配列として（つまり、n + 1個の個別のメモリブロックとして）初期化していますが、bsplibにn * n intの連続したメモリブロックであることを伝えています。割り当てまたは登録を変更する必要があります。

その結果、bsplib はまったく初期化されていないメモリを上書きします。

于 2010-12-27T16:34:28.723 に答える
1

一見無害な変更が問題を引き起こしたり、問題を解決したりするほとんどの場合、ハイゼンバグと呼ばれるものがあります。その場合、根本的な原因は実際の変更ではなく、変更は真のバグを表面化させる触媒にすぎません。

BSPlib がどのようにスレッドを実行するかは完全にはわかりませんが、がゼロでないn場合、値が初期化されていないように見えます。thread

言い換えれば、値はstring1for のみの長さに設定されますが、それはfor のスペースにthread == 0使用されます。このスペースは、たまたまスタック上にあるごみに依存しています。mallocthread !=0

于 2010-12-27T16:31:46.393 に答える
0 
    string string1((istreambuf_iterator<char>(file1)), istreambuf_iterator<char>());
    string string2((istreambuf_iterator<char>(file2)), istreambuf_iterator<char>());
    n=string1.length();
    str1= (char*)malloc(sizeof(char)*(n+1));
    str2= (char*)malloc(sizeof(char)*(n+1));
    strcpy(str1,string1.c_str());
    strcpy(str2,string2.c_str());

string2 が string1 よりも長い場合はどうなりますか? これは常に当てはまりますか？inmallocのサイズを割り当てるために使用しています。が よりも長い場合、バッファ オーバーフローが発生し、おそらくメモリ内のあらゆる種類のものを破壊します。string1str2string2string1

あなたはやるべきですn1 = string1.length(); n2 = string2.length();か？

于 2010-12-27T16:32:13.853 に答える