C言語 ポインタ型関数の扱い

Question

#include <stdio.h>
    
    #define MAX_MSGSIZE 100
    #define BUF_SIZE 10
    #define BUF_LENGTH 100
    
    char *create_cache();
    
    int main(int argc, char *argv[]) {
    
        char *Cache[BUF_SIZE];
        Cache = create_cache();
        printf("address is: %s\n", Cache[0]);
    
        return 0;
    }
    
    char *create_cache(){
        char *cache[BUF_SIZE];
        for (int size = 0; size < BUF_SIZE; ++size) {
            char buffer[BUF_LENGTH] = "i";
            cache[size] = &buffer[0];
        }
        return cache;
    }

上記のコードをコンパイルしたところ、以下のようなエラーが出てしまいました。

./main_test.c: In function ‘main’:
./main_test.c:16:11: error: assignment to expression with array type
     Cache = create_cache();
           ^
./main_test.c: In function ‘create_cache’:
./main_test.c:28:12: warning: return from incompatible pointer type [-Wincompatible-pointer-types]
     return cache;
            ^~~~~

ポインタ型関数の扱いに不慣れなのですが、どのように解決したらよいか
ご教授頂ければ幸いです。

Answer 1

オート変数の問題は、他の方が指摘しているとおりです。

char *Cache[BUF_SIZE];

は、charへのポインタの配列です。従って、char *Cacheとしましょう。

Answer 2

create_cache関数で、スタック領域に確保した変数のアドレスを返そうとしてますが、関数からreturnするとスタックが破棄されてデータもなくなりますよ。
そのアドレスを使い続けるとメモリ破壊します。
mallocでヒープ領域からメモリ確保して、そのアドレスを返すようにしてください。

Comments

[yu_2_8_2]

ありがとうございます

Answer 3

そもそも automatic変数の参照を返してはならない。