c言語 Segmentation faultについて

Question

前提・実現したいこと

コンパイルは通るのですが、プログラムを実行すると
Segmentation faultとなります。

調べてみるとmallocなどを使うべきなのかと思っていますが、
上手くかけていません。

全体の目的としてはcfgファイルから２つの数値をweight1,2として取得し、
とある計算の引数にしようと思っています。

ｃ初心者なのですが、どのような原因や対策があるのか教えて頂ければと思います。

該当のソースコード

cfgファイルから値を取得
parser.c

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

void parse_sse_ng_config(float* weight1, float* weight2)
{
    char values[100];
    char *filename;
    long lines;
    // read file
    filename="sse_ng.cfg";
    FILE* fp = fopen(filename, "r");
    lines = fread(values, sizeof(values), 1, fp);
    // read values from fp
    sscanf(values, "%f %f", weight1, weight2);
    fclose(fp);
}

parser.cからweight1,2を参照して別の関数の引数とする
cost_layer.c

#include "cost_layer.h"
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include "parser.h"

void parse_sse_ng_config(float* weight1, float* weight2);
void forward_cost_layer_gpu(cost_layer l, network net)
{//コード省略
    } else if(l.cost_type == SSE_NG){
        float weight1 = 0.0;
        float weight2 = 0.0;

        parse_sse_ng_config(&weight1, &weight2);
        sse_ng_gpu(~~~~~, weight1, weight2);
               }  else //コード省略

sse_ng.cfg
スペース区切りの数値２つ

50.0 2.0

##make コンパイル時
コンパイル時はこのような出力がでます。
もう一度makeをすると
make: Nothing to be done for 'all'.
となるので、コンパイルは通っています。

gcc -Iinclude/ -Isrc/ -DGPU -I/usr/local/cuda/include/ -DCUDNN  -Wall -Wno-unknown-pragmas -Wfatal-errors -fPIC -fopenmp -Ofast -DGPU -DCUDNN -c ./src/cost_layer.c -o obj/cost_layer.o
gcc -Wall -Wno-unknown-pragmas -Wfatal-errors -fPIC -fopenmp -Ofast -DGPU -DCUDNN -shared obj/gemm.o obj/utils.o obj/cuda.o obj/deconvolutional_layer.o obj/convolutional_layer.o ob
...............

###補足
cost_layer.cのparse_sse_ng_config(&weight1, &weight2);
をコメントアウトするとプログラムが通るので、
この部分が問題だと考えています。

parser.c内のchar values[100]の定義に問題があるのかもしれません。

補足情報（FW/ツールのバージョンなど）

ubuntu

Answer 1

まず確かめるべきは

1. ファイルは開けているか?
2. ファイルは読み込めているか?
3. weight1およびweight2は読み込めているか?

です。

1. ファイルは開けているか?

// parser.c
#include <unistd.h> // getcwd
#include <math.h>   // NaN

// `FILE* fp = fopen(filename, "r");`直後に
if(fp == NULL){
  printf("can't open %s\n", filename);
  // あんまり行儀よくないが面倒くさいので再利用
  if(getcwd(values, sizeof(values)) != NULL)
    printf("cwd: [%s]\n", values);
  fflush(stdout);
  weight1 = weight2 = NAN;
  return;
}

2. ファイルは読み込めているか?

// lines = fread(values, sizeof(values), 1, fp); 直下に
printf("%s read %d bytes, [%s]\n", filename, lines, values);
fflush(stdout);

3. weigth1, weight2は読み込めているか

// sscanf(values, "%f %f", weight1, weight2); 直下に
printf("weight1: %f, weight2: %f\n", weight1, weight2);
fflush(stdout);

まぁ、こんな感じに順に確かめていきます。

Comments

[MF_19]

ファイルの読み取りが上手く出来ていませんでした！
詳細ありがとうございます。

Answer 2

1. void parse_sse_ng_config(float* weight1, float* weight2)

このfloat* をdouble* にしよう。他のところも全部double に。

1. FILE* fp = fopen(filename, "r");

fp のNULLチェックを入れる必要あり
このままではオープンに失敗した場合、コケます

Comments

[MF_19]

回答頂きありがとうございます。