作成したプログラムでgs_report.cのエラーが出てしまう

Question

自分で作成した複数枚(添付のプログラムでは9枚)の画像を読み込んで、その平均画像を出力する簡単なプログラムなのですが動作を停止してしまいます。

gs_report.cというものに「スタック Cookie インストルメンテーション コードは、スタックベースのバッファー オーバーランを検出しました。」と表示されるのですが、意味がよく分からないのでどういったエラーなのか教えていただきたいです。

一応プログラムを実行すると平均画像の出力もされています。

#define _CRT_SECURE_NO_WARNINGS
#define _USE_MATH_DEFINES
#include<iostream>
#include<cmath>
#include<opencv2/opencv.hpp>
using namespace std;
using namespace cv;
string win_src = "src";
string win_dst = "dst";

int main(int argc, char** argv)
{
    int x, y;
    int sum[63][63] = { 0 };
    
    for (int i = -40; i <= 40; i += 10) {
        string file_src = "aa" + to_string(i) + ".pgm"; // 入力画像のファイル名
        
        Mat img_src = imread(file_src, 0); // 入力画像（グレースケール）の読み込み

        Mat img_dst;
        if (!img_src.data) {
            cout << "error" << endl;
            return -1;
        }

        for (y = 0; y < img_src.rows; y++) {
            for (x = 0; x < img_src.cols; x++) {
                sum[y][x] += img_src.at<unsigned char>(y, x);
            }
        }
    }
    
    string file_dst = "ave.pgm"; // 出力画像のファイル名
    Mat img_ave = Mat::zeros(64, 64, CV_8UC1);
    for (y = 0; y < img_ave.rows; y++) {
        for (x = 0; x < img_ave.cols; x++) {
            img_ave.at<unsigned char>(y, x) = 255;
        }
    }

    // ここに核となる処理を記述する
    for (y = 0; y < img_ave.rows; y++) {
        for (x = 0; x < img_ave.cols; x++) {
            img_ave.at<unsigned char>(y, x) = sum[y][x] / 9;
        }
    }

    imwrite(file_dst, img_ave); // 処理結果の保存

    return 0;
}

Comments

[episteme]

waitkey() の使い方、正しいですか/必要ですか?
公式ドキュメント↓読みましたか?

https://docs.opencv.org/4.5.5/d7/dfc/group__highgui.html#ga5628525ad33f52eab17feebcfba38bd7

[tkphone343]

ご指摘ありがとうございます。
ウィンドウを使っていないので消しておきます。

[episteme]

> ウィンドウを使っていないので消しておきます。

それで解決しましたか?

[tkphone343]

エラー自体は解決できていませんね…

[fana]

> 意味がよく分からないのでどういったエラーなのか

とりあえず「バッファオーバーラン」でググってみるとかすれば，ご自身のコードについて疑うべき事柄の推測が立つのではないでしょうか．

Answer 1

int sum[63][63]

そもそも何故このサイズなのか？　これは妥当なのか？
あなたが読み込んでいる画像のサイズに対して十分なサイズなのか？
外野にはわからないところではあるが，
バッファオーバーランとか言われているなら真っ先に疑うところかと思う．

で，それはそれとして，
この形だと，有効なindexの値は { 0,1,2, ..., 62 } なわけで，

Mat img_ave = Mat::zeros(64, 64, CV_8UC1);

に対して

for (y = 0; y < img_ave.rows; y++)

とかしたら，y の値は { 0,1,2, ... , 63 } （x側も同様）となるのだから，そんなループで

sum[y][x]

っていうのはまずいだろう．

Comments

[fana]

スタック上にとられているバッファオーバーランになりそうなやつって，このコードだと sum[][] くらいだよね，っていう．