コミット: 2f4eebabc66001209c7eff144138b572b4db6e9f - Tera_Term_Testing (git)

Tera Termの個人的な作業用リポジトリ

コミットメタ情報

• リビジョン: 2f4eebabc66001209c7eff144138b572b4db6e9f (tree)
• 日時: 2018-01-25 21:22:05
• 作者: IWAMOTO Kouichi <sue@iwmt...>
• コミッター: IWAMOTO Kouichi

ログメッセージ

SSH_MSG_KEY_DH_GEX_REQUEST での min のデフォルトを 2048 に引き上げた

RFC 8270対応。

SSH_MSG_KEY_DH_GEX_REQUEST での min の値は GexMinimalGroupSize で
設定可能であり、デフォルトでは 0 となっている。
GexMinimalGroupSize が 0 の時は、Minimum Limit の 1024 として扱って
いたが、RFC 8270 での推奨値に従い 2048 として扱うように変更した。
互換性で 2048 未満に設定したい場合にそなえて、Minimum Limit は
1024 のままとする。

RFC 8270 では、

| This minimum DH group size may need to be increased to 3072 for forward-looking users.

とあるので、3072 に上げてもいいのかも

git-svn-id: svn+ssh://svn.osdn.net/svnroot/ttssh2/trunk@7028 f5f01b69-1e22-0410-acbf-894ab4bd6246

変更サマリ

• modified: doc/en/html/about/history.html
• modified: doc/ja/html/about/history.html
• modified: ttssh2/ttxssh/kex.h
• modified: ttssh2/ttxssh/ssh.c

差分

--- a/doc/en/html/about/history.html
+++ b/doc/en/html/about/history.html
@@ -2970,6 +2970,15 @@ SYNOPSIS:
 
 <h3><a name="ttssh_2.84">2018.02.28 (Ver 2.84) not released</a></h3>
 <ul class="history">
+  <li>Changes
+    <ul>
+      <li>The minimum group size of the Diffie-Hellman group exchange key exchange method is increased to 2048. (RFC 8270)
+        <ul>
+          <li>To change to the previous behavior, set the GexMinimalGroupSize entry in teraterm.ini file to 1024.</li></li>
+        </ul></li>
+    </ul>
+  </li>
+
   <li>Bug fixes
     <ul>
       <!--li>バージョン情報ダイアログのサーバホスト鍵の指紋に、同一サーバで接続に使用されていない方式の鍵が表示されることがある問題を修正した。</li-->

--- a/doc/ja/html/about/history.html
+++ b/doc/ja/html/about/history.html
@@ -2976,6 +2976,15 @@
 
 <h3><a name="ttssh_2.84">2018.02.28 (Ver 2.84) not released</a></h3>
 <ul class="history">
+  <li>変更
+    <ul>
+      <li>Diffie-Hellman group exchane 鍵交換方式での最小グループサイズを 2048 に変更した。(RFC 8270)
+        <ul>
+          <li>以前の動作に戻すには、teraterm.ini の GexMinimalGroupSize エントリを 1024 にする。</li>
+        </ul></li>
+    </ul>
+  </li>
+
   <li>バグ修正
     <ul>
       <li>バージョン情報ダイアログのサーバホスト鍵の指紋に、同一サーバで接続に使用されていない方式の鍵が表示されることがある問題を修正した。</li>

--- a/ttssh2/ttxssh/kex.h
+++ b/ttssh2/ttxssh/kex.h
@@ -31,6 +31,10 @@
 // SSH_MSG_KEY_DH_GEX_REQUEST での min, n, max がとり得る範囲の上限/下限 (RFC 4419)
 #define GEX_GRP_LIMIT_MIN   1024
 #define GEX_GRP_LIMIT_MAX   8192
+// GexMinimalGroupSize が 0 (デフォルト(未設定)) だった時に min に使う値
+// RFC 8270 で min の最低値が 2048 に引き上げられたが、互換性の為に GEX_GRP_LIMIT_MIN
+// を引き上げるのではなくて、デフォルトの値を変更する
+#define GEX_GRP_DEFAULT_MIN 2048
 
 DH *dh_new_group1(void);
 DH *dh_new_group14(void);

--- a/ttssh2/ttxssh/ssh.c
+++ b/ttssh2/ttxssh/ssh.c
@@ -5375,9 +5375,15 @@ error:;
 
 
 
-//
-// DH-GEX (RFC 4419)
-//
+/*
+ * DH-GEX (RFC 4419)
+ *
+ * SSH_MSG_KEY_DH_GEX_REQUEST:
+ *   byte    SSH_MSG_KEY_DH_GEX_REQUEST (34)
+ *   uint32  min, minimal size in bits of an acceptable group
+ *   uint32  n, preferred size in bits of the group the server will send
+ *   uint32  max, maximal size in bits of an acceptable group
+ */
 
 static void SSH2_dh_gex_kex_init(PTInstVar pvar)
 {
@@ -5391,17 +5397,31 @@ static void SSH2_dh_gex_kex_init(PTInstVar pvar)
 		goto error;
 	}
 
-	// サーバが保証すべき最低限のビット数を求める（we_needはバイト）。
-	if (pvar->settings.GexMinimalGroupSize < GEX_GRP_LIMIT_MIN) {
+	// サーバに要求する group size の min, n(preferred), max を決定する。
+	if (pvar->settings.GexMinimalGroupSize == 0) {
+		// 0 (未設定) だった時は最新の推奨値を使う
+		min = GEX_GRP_DEFAULT_MIN;
+	}
+	else if (pvar->settings.GexMinimalGroupSize < GEX_GRP_LIMIT_MIN) {
 		min = GEX_GRP_LIMIT_MIN;
+		logprintf(LOG_LEVEL_NOTICE,
+			__FUNCTION__ ": small GexMinimalGroupSize is too small (%d), use minimum limit (%sd)",
+			pvar->settings.GexMinimalGroupSize, GEX_GRP_LIMIT_MIN);
 	}
 	else if (pvar->settings.GexMinimalGroupSize > GEX_GRP_LIMIT_MAX) {
 		min = GEX_GRP_LIMIT_MAX;
+		logprintf(LOG_LEVEL_NOTICE,
+			__FUNCTION__ ": small GexMinimalGroupSize is too larse (%d), use maximum limit (%sd)",
+			pvar->settings.GexMinimalGroupSize, GEX_GRP_LIMIT_MAX);
 	}
 	else {
 		min = pvar->settings.GexMinimalGroupSize;
 	}
+
+	// max は常に上限いっぱい
 	max = GEX_GRP_LIMIT_MAX;
+
+	// preferred は使用する各暗号要素の鍵長/ブロック長のうち、最大のものを使う
 	bits = dh_estimate(pvar->we_need * 8);
 	if (bits < min) {
 		bits = min;