式に名前を付けて間違いを防止する

sakura-editor · Apr 25, 2020 · 1b4f138 · 1b4f138
1 parent 9bc1009
commit 1b4f138
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/sakura_core/typeprop/CImpExpManager.cpp b/sakura_core/typeprop/CImpExpManager.cpp
@@ -661,11 +661,14 @@ bool CImpExpRegex::Import( const wstring& sFileName, wstring& sErrMsg )
 				}
 				if( k != -1 )	/* 3文字カラー名からインデックス番号に変換 */
 				{
-					if( 0 < MAX_REGEX_KEYWORDLISTLEN - keywordPos - 1 ){
+					// pKeywordに書き込める残りサイズ(NUL終端分を含む)
+					const size_t cchAvailableSize = MAX_REGEX_KEYWORDLISTLEN - keywordPos;
+
+					if( 0 < cchAvailableSize - 1 ){
 						regexKeyArr[count].m_nColorIndex = k;
-						wcsncpy_s( &pKeyword[keywordPos], std::min<size_t>(MAX_REGEX_KEYWORDLEN, MAX_REGEX_KEYWORDLISTLEN - keywordPos - 1), p, _TRUNCATE );
+						wcsncpy_s( &pKeyword[keywordPos], std::min<size_t>(MAX_REGEX_KEYWORDLEN, cchAvailableSize), p, _TRUNCATE );
 						count++;
-						keywordPos += wcsnlen( &pKeyword[keywordPos], MAX_REGEX_KEYWORDLISTLEN - keywordPos ) + 1;
+						keywordPos += wcsnlen( &pKeyword[keywordPos], cchAvailableSize) + 1;
 					}else{
 						sErrMsg = LS(STR_IMPEXP_REGEX2);
 					}