麻雀の役判定でエラー

Question

int uchiwake[10];
bool sansyokudoujun(void)
/*
	三色同順
*/
{
	cout << "sansyokudoujun" << endl;
	int tmp[34];
	memset(tmp, 0, sizeof(tmp));

	for (int i = 0; i < 10; i += 2)
	{
		switch (uchiwake[i])
		{
		case SYUNTU:
		case CHII:
			tmp[uchiwake[i + 1]]++;
			break;
		default:
			break;
		}
	}
	cout << "sansyokudoujun2" << endl;
	for (int i = 0; i < 9; i++)
	{
		if (tmp[i] && tmp[i + 9] && tmp[i + 18]) return true;
	}
	cout << "sansyokudoujun3" << endl;
	return false;
}

c++で麻雀の役判定をしています。
たまに**Stack around the variable 'tmp' was corrupted.**とエラーを吐いてしまいます。
uchiwakeの中身は偶数にメンツの状態、奇数に牌番号があらかじめ入っています。
順子やチーの場合そのメンツの中で一番低い牌番号が入っています。
SYUNTUやCHIIはenumです。
pybind11でビルドしてつかっているのでどの行数でのエラーかまではわかりませんが、
エラーのとき、cout << "sansyokudoujun3" << endl;までは出力できています。
どこが原因なのかわかりません。ご回答お待ちしております。

Answer 1

tmp[uchiwake[i + 1]]++;

uchiwake[i + 1]の内容がサイズを超えてるんでしょうね

Comments

[Owan]

回答ありがとうございます。
牌番号は27以上は字牌で順子、チーの情報にはならないので
牌番号が27以上のものはプログラム上評価されないはずなのですが、違いますか？

[Y.H.]

iは9まで。i+1は？

[y_waiwai]

あなたのいう「はず」という言葉が本当ならエラーにならないわけですが、実際はどうなんでしょうか。
uchiwakeのナカミが提示されていない以上、私に聞かれてもなんとも言えません

[Owan]

Y.Hさんへ　
コメントありがとうございます。このfor文ではiは8まででi+1は実際9までしか参照されないと思います。

[Owan]

y_waiwaiさんへ
そうですよね。これから、uchiwakeの中身を出力して確認したいと思います。

[Owan]

y_waiwaiさんへ
今中身を確認したところ範囲外の34が入っていました。
y_waiwaiさんのおっしゃる通りでした。お答えいただきありがとうございました。