Segmentation faultエラーについて

Question

前提

CでWindowsのコンソールアプリを作っていたところ、エラーが出たので、そのことについて。

実現したいこと

• エラーが出ないようにしたいです

発生している問題・エラーメッセージ

該当のソースコード

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

int main(void){
    printf("condole settion started!\n");
    getch();
    FILE *fn;
    fn = fopen("C:/gcc/Project/data.dtbs", "r");
    if(fn = NULL){
        printf("ERROR:CANNOT OPEN THE FILE:C:/gcc/Project/data.dtbs");
        exit(0);
    }
    char ch[4336620];
    int cnt = 0;
    while( (ch[cnt] = fgetc(fn)) != EOF){
        cnt++;
    }
    char *str_start;
    str_start = strstr(ch, "<p class=\"weather-telop\">");
    printf("DeBug:%s", str_start);

    getch();
    fclose(fn);
    return 0;
}

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

Windows 10 Pro 32bit
Virtual Studio Code 1.67.2

Comments

[thkana]

> エラーが出ないようにしたいです
で、質問としては、なにを求めていますか?
「回答者は問題を解決したコードを示せ」ということですか?

Answer 1

今どきのGB単位のメモリが載ったPCでたかだか4MBちょいの配列が作れないのはおかしな話ですが、Visualtudioだと、プログラムを実行したときのスタックサイズが1MBくらいしかないようです。そのためauto変数が大きいと実行時エラーになります。

この参考記事にあるようにスタックサイズを広げる、自己解決されたようにmallocで領域を確保する、autoではなく静的に確保する、といったあたりが対策です。

Answer 2

どうやら

char ch[4336620];

の部分が悪かったようです。
そこの所を

char *ch = malloc(4336620);

として、それに伴って変える場所を変えると、エラーが解決しました。

Comments

[thkana]

そもそも4336620という値は適切なのでしょうか?

[0266st]

これは開こうとしているファイルの文字数を求めたものに、多少の余分を追加したものなので多分適切であると思います。

[jimbe]

>開こうとしているファイルの文字数を求めたもの
そのような場合は、固定値では無く、プログラムでファイルのサイズを求めてその分のメモリを malloc する形にするほうが良いと思います。

Answer 3

ローカル変数で
char ch[4336620];
という配列のサイズが無茶だと思います。32bitシステムですし...

Comments

[0266st]

ありがとうございました。