VScodeでC言語を実行すると「'hello' accessing 1024 bytes in a region of size 7 [-Wstringop-overflow=]」が表示される

Question

前提

C言語をVScodeを利用して勉強しています。
利用OSはWindows 11 64bitです。

以下に記す該当のソースコードを実行すると[出力]は正常ですが、[問題]の項目にエラーメッセージが表示されました。

実現したいこと

• 問題の項目のエラーが表示されないようにしたい

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

'hello' accessing 1024 bytes in a region of size 7 [-Wstringop-overflow=]

該当のソースコード

#include <stdio.h>

typedef char String[1024];

void hello(String name) {
    printf("%sさん、こんにちは\n", name);
}

int main(void) {
    hello("田中");
    return 0;
}

試したこと

”田中”の文字を"t"に変更した場合、エラーメッセージの7が2に変わりました。

typedef char String[1024]をtypdef char String[64]に変更するとエラーメッセージの1024が64に変わりました。

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

★インストール済みの拡張機能

{
    "configurations": [
        {
            "name": "MinGW",
            "includePath": [
                "${workspaceFolder}/**",
                "C:\\gcc\\mingw64\\include"
            ],
            "defines": [
                "_DEBUG",
                "UNICODE",
                "_UNICODE"
            ],
            "windowsSdkVersion": "10.0.22000.0",
            "compilerPath": "C:/gcc/mingw64/bin/gcc.exe",
            "cStandard": "c17",
            "cppStandard": "c++17",
            "intelliSenseMode": "windows-gcc-x64"
        }
    ],
    "version": 4
}

{
  "configurations": [
    {
      "name": "C/C++: gcc.exe アクティブ ファイルのビルドとデバッグ",
      "type": "cppdbg",
      "request": "launch",
      "program": "${fileDirname}\\${fileBasenameNoExtension}.exe",
      "args": [],
      "stopAtEntry": false,
      "cwd": "${fileDirname}",
      "environment": [],
      "externalConsole": false,
      "MIMode": "gdb",
      "miDebuggerPath": "C:\\gcc\\mingw64\\bin\\gdb.exe",
      "setupCommands": [
        {
          "description": "gdb の再フォーマットを有効にする",
          "text": "-enable-pretty-printing",
          "ignoreFailures": true
        },
        {
          "description": "逆アセンブリ フレーバーを Intel に設定",
          "text": "-gdb-set disassembly-flavor intel",
          "ignoreFailures": true
        }
      ],
      "preLaunchTask": "C/C++: gcc.exe アクティブなファイルのビルド"
    },
  ],
  "version": "2.0.0"
}

{
    "tasks": [
        {
            "type": "cppbuild",
            "label": "C/C++: gcc.exe アクティブなファイルのビルド",
            "command": "C:\\gcc\\mingw64\\bin\\gcc.exe",
            "args": [
                "-fdiagnostics-color=always",
                "-g",
                "${file}",
                "-o",
                "${fileDirname}\\${fileBasenameNoExtension}.exe"
            ],
            "options": {
                "cwd": "${fileDirname}"
            },
            "problemMatcher": [
                "$gcc"
            ],
            "group": {
                "kind": "build",
                "isDefault": true
            },
            "detail": "デバッガーによって生成されたタスク。"
        }
    ],
    "version": "2.0.0"
}

Answer 1

関数 hello() がサイズ1024のchar配列を引数として要求しているのに、サイズ1024無い文字列定数を渡しているので危険に見えるということです。

処理内容によりますが、この hello() の実装なら、hello() の引数は const char* にすべきでしょう。
または、main() で String型の変数を文字列定数で初期化してからhello() に渡しましょう。

Comments

[int32_t]

間違ってはないですが、「引数は const char* に」か「 String型の変数を文字列定数で初期化してからhello() に渡し」のどちらかだけでいいです。

[usan]

追加でご回答ありがとうございます。

「引数はconst char*に」の変更は以下のようにすると上手くいきました。

‘‘‘
#include <stdio.h>

//typedef char String[1024];

void hello(const char* name) {
printf("%sさん、こんにちは\n", name);
}

int main(void) {
hello("田中");
return 0;
}
‘‘‘

しかし「 String型の変数を文字列定数で初期化してからhello() に渡し」の方法はわかりませんでした。

[usan]

VScodeを再起動して以下を試すとエラーが出ずに上手く動作しました！

‘‘‘
#include <stdio.h>

typedef char String[1024];

void hello(String name) {
printf("%sさん、こんにちは\n", name);
}

int main (void) {

String name = "田中";
hello(name);

return 0;
}
‘‘‘

再起動しなければうまくいかなかった理由は不明です。

ご回答いただきありがとうございました！

[y_waiwai]

> typedef char String[1024];

なにかで聞きかじってこういうことしてるんでしょうけど、初心者のうちはこういう余計なことはしないほうがよろしいかと。
言語の文法を学ぶ上でいいことはありません

[usan]

>なにかで聞きかじってこういうことしてるんでしょうけど、

まさにその通りで、2021年12月11日 初版発行の「ス〇キリわかるC言語入門 第2版」に、実際にプログラムの一例例として記載されていた内容をそのまま写経して実行したところエラーがでたために質問したんですよね。

C言語の初学者であるため、今の私にはこれがどんな時にどういう理由から余計なことになるのか理解できないでいます。

[usan]

あれから本書を読み進めていくことでC言語にいわゆるString型が存在せず、あくまで「入門をスムーズに進めるための苦肉の策であった」ということを理解できました。

本書のやり方が最も良い方法であったのかはわかりかねますが、ポインタの理解が進んだ現在では本書の方針に感謝しております。