SonarCloudにBugだと言われているGetExistPathWの範囲外アクセスを修正する

[berryzplus]

PR の目的

SonarCloudの警告により発覚した潜在不具合（？）を修正します。

カテゴリ

• 不具合修正

PR の背景

SonarCloudにより以下のBugsレベル警告が検出されています。

Out of bound memory access (accessed memory precedes memory block)
https://sonarcloud.io/project/issues?id=sakura-editor_sakura&issues=AWzTrq7uQSy0V0BNOlDQ&open=AWzTrq7uQSy0V0BNOlDQ

sakura/sakura_core/util/file.cpp

Lines 840 to 844 in 8f92e0b

	/* ↓ルートディレクトリを引っかけるための処理 */
	if( ( *pw == L'\\' )&&( *(pw-1) == L':' ) ){ /* C:\ とかの \ っぽい */
	* (pw+1) = L'\0'; /* \ の後ろの位置を文字列の終端にする。 */
	if( _waccess(po,0) == 0 ) break; /* 有効なパス文字列が見つかった */
	}

このコードは pw = ps + 0 のときにも通るので、 ノーチェックで pw - 1 にアクセスするのはマズいです。
※psには書き込み先バッファの先頭アドレスが入ってるっぽいです。

PR のメリット

SonarCloudのBugsレベル警告が1つ解消します。

PR のデメリット (トレードオフとかあれば)

とくにありません。

仕様・動作説明

pw - 1 にアクセスする前にチェックを入れます。

PR の影響範囲

テスト内容

ビルド確認のみです。

関連 issue, PR

#1504

参考資料

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[AppVeyorBot]

✅ Build sakura 1.0.3565 completed (commit d718db0099 by @berryzplus)

[berryzplus]

レビューありがとうございます。マージしちゃいます。