Merge pull request sakura-editor#947 from berryzplus/feature/segfault…

…_test_for_MSVC_Release

MSVCリリースビルドで失敗するテストを修正する

tests/unittests/test-mydevmode.cpp

@@ -258,22 +258,13 @@ TEST(MYDEVMODETest, operatorNotEqual)
 	EXPECT_EQ(myDevMode, value);
 }
 
-/* アクセス不可のメモリ領域にアクセスしても、例外が発生しない事象があるので、
- * 当面MSVCのリリース版では以下のテストを実行しない。
- *
- * 参考: https://github.com/google/googletest/blob/9d4cde44a4a3952cf21861f9370b3bed9265dfd7/googletest/docs/advanced.md#temporarily-disabling-tests
- */
-#if defined(_DEBUG) || defined(__MINGW64__)
 /*!
  * @brief 等価比較演算子が一般保護違反を犯さないことを保証する非機能要件テスト
  *
  *  通常、ここまでやる必要はないが、修正の理由が「安全のため」なので、
  *  実際にどういうケースで一般保護例外違反となるか、コード的に発生させる方法の共有を兼ねて実装したもの。
  */
 TEST(MYDEVMODETest, StrategyForSegmentationFault)
-#else
-TEST(MYDEVMODETest, DISABLED_StrategyForSegmentationFault)
-#endif /* if defined(_DEBUG) || defined(__MINGW64__) */
 {
 	// システムのページサイズを取得する
 	SYSTEM_INFO systemInfo = { 0 };
@@ -286,14 +277,19 @@ TEST(MYDEVMODETest, DISABLED_StrategyForSegmentationFault)
 
 	// 仮想メモリ範囲を予約する。予約時点では全体をNOACCESS指定にしておく。
 	LPVOID memBlock1 = ::VirtualAlloc(NULL, allocSize, MEM_RESERVE, PAGE_NOACCESS);
-	assert(memBlock1);
+	EXPECT_TRUE(memBlock1 != NULL);
+
+	// 仮想メモリ全域をコミット(=確保)する。
+	wchar_t* buf1 = static_cast<wchar_t*>(::VirtualAlloc(memBlock1, allocSize, MEM_COMMIT, PAGE_READWRITE));
+	EXPECT_TRUE(buf1 != NULL);
 
-	// 仮想メモリを1ページ分だけコミット(=確保)する。2ページ目はNOACCESSのまま。
-	wchar_t* buf1 = static_cast<wchar_t*>(::VirtualAlloc(memBlock1, pageSize, MEM_COMMIT, PAGE_READWRITE));
-	assert(buf1);
+	// 確保したメモリ全域をASCII文字'a'で埋める
+	::wmemset(buf1, L'a', pageSize);
 
-	// 確保したメモリ領域をASCII文字'a'で埋める
-	::wmemset(buf1, L'a', pageSize / sizeof(wchar_t));
+	// 2ページ目の保護モードをNOACCESSにする。
+	DWORD flOldProtect = 0;
+	volatile BOOL retVirtualProtect = ::VirtualProtect((char*)buf1 + pageSize, pageSize, PAGE_NOACCESS, &flOldProtect);
+	EXPECT_TRUE(retVirtualProtect);
 
 	// メモリデータをテスト対象型にマップする。実態として配列のように扱えるポインタを取得している。
 	MYDEVMODE* pValues = reinterpret_cast<MYDEVMODE*>(buf1);
@@ -309,7 +305,9 @@ TEST(MYDEVMODETest, DISABLED_StrategyForSegmentationFault)
 	 * 想定結果：一般保護違反で落ちる
 	 * 備考：例外メッセージは無視する(例外が起きたことが検知できればよいから。)
 	 */
-	ASSERT_DEATH({ ::_tcscmp(pValues[0].m_szPrinterDeviceName, pLargeStr); }, ".*");
+	volatile int ret = 0;
+	ASSERT_DEATH({ ret = ::_tcscmp(pValues[0].m_szPrinterDeviceName, pLargeStr); }, ".*");
+	(void)ret;
 
 	// 等価比較演算子を使った場合には落ちないことを確認する
 	EXPECT_TRUE(pValues[0] == pValues[1]);