ノート

[BUG] 8366058: Outdated comment in WinCAPISeedGenerator

Windows 95 についての古いコメントを削除しました。

概要#

　Windows 向けの​ WinCAPISeedGenerator.c では，​MS CryptoAPI から​乱数シードを​取得する​ nativeGenerateSeed が​定義されています1。​ この​メソッドの​直前には，​以下のように​ Windows 95 の​初期バージョンに​関する​コメントが​残っていました2。

/*
 * Get a random seed from the MS CryptoAPI. Return true if successful, false
 * otherwise.
 *
 * Some early versions of Windows 95 do not support the required functions.
 * Use runtime linking to avoid problems.
 *
 */

　しかし，​JDK では​既に​ Windows 95 を​サポートしていません。​ その​ため，​現在の​実装を​理解する​うえで​この​コメントは​有用ではなく，​むしろ​「Windows 95 への​配慮が​まだ​必要なのか」と​いう​誤解を​招く​恐れが​ありました。

　​今回の​修正では，​この​古くなった​ Windows 95 に​ついての​コメントだけを​削除しました3。​ コードの​挙動は​変更していないため，​テストを​追加する​種類の​変更ではなく，​JBS には​ noreg-doc を​付与しました。

2025/11/02 JBS で​発見#

　本バグを​ JBS で​発見し，​対象ファイルを​確認しました。​ 問題の​コメントは​ nativeGenerateSeed の​説明コメントの​一部と​して​残っていましたが，​現在サポートされる​ Windows とは​関係が​ありません。

　修正内容は​コメントの​削除のみです。​ 以下の​ 4 行を​削除し，​メソッド説明だけを​残しました4。

- *
- * Some early versions of Windows 95 do not support the required functions.
- * Use runtime linking to avoid problems.
- *

　PR 作成直後は​ core-libs ラベルが​自動で​付与されましたが，​対象は​セキュリティ系の​ネイティブ実装である​ため，​Alan Bateman5 氏に​より​ security ラベルへ​変更されました。

2026/01/05 レビュー#

　Sean Mullan6 氏に​レビューを​いただき，​Approve されました。​ また，​あわせて​ JBS に​ noreg-doc ラベルを​付与するようコメントを​いただき，​追加しました。

　問題ない​ことを​確認して，​統合リクエストを​行いました。

2026/01/05 統合#

　Mullan6 氏に​ /sponsor を​していただき eee58547 と​して​ JDK へ​統合されました。

Footnotes#

  1. https://github.com/openjdk/jdk/blob/e8a1a8707ee6192c85ac62a2a51c815e07613c38/src/java.base/windows/native/libjava/WinCAPISeedGenerator.c#L31-L40 ↩

  2. https://github.com/openjdk/jdk/blob/e8a1a8707ee6192c85ac62a2a51c815e07613c38/src/java.base/windows/native/libjava/WinCAPISeedGenerator.c#L34-L37 ↩

  3. https://github.com/openjdk/jdk/pull/28105 ↩

  4. https://github.com/openjdk/jdk/pull/28105/files ↩

  5. https://openjdk.org/census#alanb ↩

  6. https://openjdk.org/census#mullan ↩ ↩2

  7. https://github.com/openjdk/jdk/commit/eee58545c8342fc39b3bec5b38da6c01d92d05f2 ↩