Segmentation fault: 11を解決する方法に関して

Question

実行時に Segmentation fault: 11
というエラーが出力されます。

アクセスできないメモリのアドレスにアクセスする際にエラーが出力されるというのは分かったのですが、どう直してよいのかわかりません。

#include<stdio.h>
#include<stdlib.h>
#include<math.h>
#include<string.h>
#include<time.h>

#define N 64

int main(int argc, char *argv[])
{
int i;
double X,Y;
double Z1,Z2;
double P;
double D1,D2,D3;
double d1,d2;
double min;
double G1;
double G2;
double i1,i2;
double dn1,dn2;
double error, BER;
FILE *fp1;

double ave=atof(argv[1]);
double dis=atof(argv[2]);

char f_name[64];
  printf("格納するファイル名を入力してください:");
    scanf("%s",f_name);

    if((fp1=fopen(f_name,"w"))==NULL)
      {
printf("ファイルが開けませんでした\n");
}

srand((unsigned)time (NULL));
for(P=0; P<=1; P=P+0.05)
{
error=0;
for(i=1; i<=(10000000/3); i++)
{
X=(double)rand()/(RAND_MAX);
 printf("%f ",X);
Y=(double)rand()/(RAND_MAX);
 printf("%f ",Y);

Z1=sqrt(-2.0*log(X))*sin(2*M_PI*Y);
Z2=sqrt(-2.0*log(X))*cos(2*M_PI*Y);

G1 =ave + dis*Z1;
G2 =ave + dis*Z2;

D1=(double)rand()/(RAND_MAX);
D2=(double)rand()/(RAND_MAX);
D3=(double)rand()/(RAND_MAX);

if(D1>=0.5)
{
D1=1.0;
}
else
{
D1=0.0;
}

if(D2>=0.5)
{
D2=1.0;
D3=D2;
}
else
{
D2=0.0;
D3=D2;
}

i1 = P*D1+(1-P)*D2+G1;
i2 = P*D2+(1-P)*D3+G2;

if(D1==0)
{
d1=sqrt(pow(i1,2.0)+pow(i2,2.0));

d2=sqrt(pow((1-P)-i1,2.0)+pow(1-i2,2.0));
}

if(D1==1)
{
d1=sqrt(pow(P-i1,2.0)+pow(i2,2.0));

d2=sqrt(pow(1-i1,2.0)+pow(1-i2,2.0));
}

min=d1;

if(d2<min) min = d2;

if(min==d1) dn1=0, dn2 = 0;

if(min==d2) dn1 = 1, dn2 = 1;

if(dn1!=D2) error++;
}

BER=error/(10000000/3);

fprintf(fp1, "%lf %e \n",P,BER);
}
fclose(fp1);

return 0;
}

Comments

[int32_t]

デバッガ上で実行して、どこで segmentation fault になるか調べてください。
あと、人にコードを見せるときは clang-format やエディタの機能で整形しましょう。

[tatsu99]

どのような値を起動パラメータで指定したとき、 segmentation fault になりますか。

[KYUURI]

パラメータを入力する前に　segmentation fault　になります。

Answer 1

double ave=atof(argv[1]);
double dis=atof(argv[2]);

ここで使用しているatof関数に渡す引数はアドレスです。
質問者様のコードですとargv[1]の中身を渡していますので、アクセスできないメモリのアドレスにアクセスしようとしてしまい今回のエラーが出ていると思います。

以下のページを読めば修正方法はわかると思います。
配列とポインタとアドレス

Comments

[KYUURI]

こちらのページを参考にして修正します。
ありがとうございます。

Answer 2

質問に書いてある情報では絞りきれないので一般論ですが、
・配列の扱い(添字の範囲は正しいか)
・ポインタの扱い(ポインタが指している先は確保された領域か)
・(初心者の場合)scanfの&のつけ忘れ (付ける場合と付けない場合の区別)
を徹底的にチェックしてください。それで8割は解決すると思います。

今回で言えば

  double ave=atof(argv[1]);
  double dis=atof(argv[2]);

  char f_name[64];
  printf("格納するファイル名を入力してください:");
  scanf("%s",f_name);
  if((fp1=fopen(f_name,"w"))==NULL)
  {
    printf("ファイルが開けませんでした\n");
  }

この辺は大丈夫、と言い切れますか? (なお、むやみにいじって「当たりを狙う」というやり方はしないように...)

Comments

[KYUURI]

アクセスできないメモリのアドレスにアクセスしようとしており、エラーが出力されていることを理解しました。