Skip to content

CopyDirDirのバッファオーバーフロー対策#2501

Merged
berryzplus merged 7 commits into
sakura-editor:masterfrom
berryzplus:feature/fix_overflow
Jul 2, 2026
Merged

CopyDirDirのバッファオーバーフロー対策#2501
berryzplus merged 7 commits into
sakura-editor:masterfrom
berryzplus:feature/fix_overflow

Conversation

@berryzplus

Copy link
Copy Markdown
Contributor

PR対象

  • アプリ(サクラエディタ本体)
  • テストコード

カテゴリ

  • 改善

PR の背景

仕様・動作説明

  • CopyDirDir関数をバッファサイズを渡せるよう引数変更します。
  • 内部で呼び出しているAddLastYenFromDirectoryPath関数にも同じ問題があるため、同様に対処します。
  • tests1.exeについて保留していた既知の問題に対する修正の一部を適用します。

PR の影響範囲

テスト内容

関連 issue, PR

参考資料

メモリ未確保のときにパラメーター検証例外が出てしまう。
古いverのgmockではデフォルト動作に関数ポインターを渡す際にInvokeを挟む必要があったが、最新verではdeprecatedになっている。
GitHub Actionsはパイプ出力を見るので化けない
不要な分岐と処理内容に一致しないコメントを除去する。
@github-actions

Copy link
Copy Markdown

Test Results

1 103 tests  +5   1 103 ✅ +5   5m 8s ⏱️ +20s
  104 suites +1       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 9c9ede2. ± Comparison against base commit bca1be1.

@sonarqubecloud

Copy link
Copy Markdown

@berryzplus berryzplus marked this pull request as ready for review June 28, 2026 23:59
@berryzplus

Copy link
Copy Markdown
Contributor Author

本件、勝手にやります、と宣言してなかったので宣言します。

@berryzplus berryzplus merged commit 1aa9bec into sakura-editor:master Jul 2, 2026
13 checks passed
@berryzplus berryzplus deleted the feature/fix_overflow branch July 2, 2026 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants