[Ttssh2-commit] [8554] RSA署名が不正でも正しい署名だとみなす可能性が有ったのを修正

Back to archive index
scmno****@osdn***** scmno****@osdn*****
2020年 2月 20日 (木) 03:03:24 JST


Revision: 8554
          https://osdn.net/projects/ttssh2/scm/svn/commits/8554
Author:   doda
Date:     2020-02-20 03:03:24 +0900 (Thu, 20 Feb 2020)
Log Message:
-----------
RSA署名が不正でも正しい署名だとみなす可能性が有ったのを修正

MFT: r8553

問題:
  メモリが不足気味な環境な時、KEXのRSAホスト鍵での署名検証で不正な
  署名を正しい署名だと判断する可能性が有った。

原因:
  malloc() が失敗した時に return している値(1)が、正しい署名の時と
  同じだった

対処:
  malloc() が失敗した時は、署名が不正な時と同じ値(0)を返すようにした

Revision Links:
--------------
    https://osdn.net/projects/ttssh2/scm/svn/commits/8553

Modified Paths:
--------------
    branches/4-stable/doc/en/html/about/history.html
    branches/4-stable/doc/ja/html/about/history.html
    branches/4-stable/ttssh2/ttxssh/key.c

-------------- next part --------------
Modified: branches/4-stable/doc/en/html/about/history.html
===================================================================
--- branches/4-stable/doc/en/html/about/history.html	2020-02-19 17:59:21 UTC (rev 8553)
+++ branches/4-stable/doc/en/html/about/history.html	2020-02-19 18:03:24 UTC (rev 8554)
@@ -31,6 +31,27 @@
 
 <h2 id="teraterm">Tera Term</h2>
 
+<h3 id="teraterm_4.106">2020.xx.xx (Ver 4.106)</h3>
+<ul class="history">
+  <!-- li>Changes
+    <ul>
+      <li></li>
+    </ul>
+  </li -->
+
+  <!-- li>Bug fixes
+    <ul>
+      <li></li>
+    </ul>
+  </li -->
+
+  <li>Misc
+    <ul>
+      <li>upgraded TTSSH to <a href="#ttssh_2.92">2.92</a>.</li>
+    </ul>
+  </li>
+</ul>
+
 <h3 id="teraterm_4.105">2019.12.7 (Ver 4.105)</h3>
 <ul class="history">
   <li>Changes
@@ -3241,6 +3262,28 @@
 
 <h2 id="ttssh">TTSSH</h2>
 
+<h3 id="ttssh_2.92">2020.xx.xx (Ver 2.92)</h3>
+<ul class="history">
+  <!-- li>Changes
+    <ul>
+      <li></li>
+    </ul>
+  </li -->
+
+  <li>Bug fixes
+    <ul>
+      <li>システムのメモリが不足している時、鍵交換時のホスト鍵による署名検証で不正な署名を正しい物として扱う可能性が有
+った問題を修正した。</li>
+    </ul>
+  </li>
+
+  <!-- li>Misc
+    <ul>
+      <li></li>
+    </ul>
+  </li -->
+</ul>
+
 <h3 id="ttssh_2.91">2019.12.7 (Ver 2.91)</h3>
 <ul class="history">
   <li>Changes

Modified: branches/4-stable/doc/ja/html/about/history.html
===================================================================
--- branches/4-stable/doc/ja/html/about/history.html	2020-02-19 17:59:21 UTC (rev 8553)
+++ branches/4-stable/doc/ja/html/about/history.html	2020-02-19 18:03:24 UTC (rev 8554)
@@ -31,6 +31,27 @@
 
 <h2 id="teraterm">Tera Term</h2>
 
+<h3 id="teraterm_4.106">2020.xx.xx (Ver 4.106)</h3>
+<ul class="history">
+  <!-- li>\x95ύX
+    <ul>
+      <li></li>
+    </ul>
+  </li -->
+
+  <!-- li>\x83o\x83O\x8FC\x90\xB3
+    <ul>
+      <li></li>
+    </ul>
+  </li -->
+
+  <li>\x82\xBB\x82̑\xBC
+    <ul>
+      <li><a href="#ttssh_2.92">TTSSH(2.92)</a>\x82֍\xB7\x82\xB5\x91ւ\xA6\x82\xBD\x81B</li>
+    </ul>
+  </li>
+</ul>
+
 <h3 id="teraterm_4.105">2019.12.7 (Ver 4.105)</h3>
 <ul class="history">
   <li>\x95ύX
@@ -3247,6 +3268,27 @@
 
 <h2 id="ttssh">TTSSH</h2>
 
+<h3 id="ttssh_2.92">2020.xx.xx (Ver 2.92)</h3>
+<ul class="history">
+  <!-- li>\x95ύX
+    <ul>
+      <li></a>
+    </ul>
+  </li -->
+
+  <li>\x83o\x83O\x8FC\x90\xB3
+    <ul>
+      <li>\x83V\x83X\x83e\x83\x80\x82̃\x81\x83\x82\x83\x8A\x82\xAA\x95s\x91\xAB\x82\xB5\x82Ă\xA2\x82鎞\x81A\x8C\xAE\x8C\xF0\x8A\xB7\x8E\x9E\x82̃z\x83X\x83g\x8C\xAE\x82ɂ\xE6\x82鏐\x96\xBC\x8C\x9F\x8F؂ŕs\x90\xB3\x82ȏ\x90\x96\xBC\x82𐳂\xB5\x82\xA2\x95\xA8\x82Ƃ\xB5\x82Ĉ\xB5\x82\xA4\x89”\\x90\xAB\x82\xAA\x97L\x82\xC1\x82\xBD\x96\xE2\x91\xE8\x82\xF0\x8FC\x90\xB3\x82\xB5\x82\xBD\x81B</li>
+    </ul>
+  </li>
+
+  <!-- li>\x82\xBB\x82̑\xBC
+    <ul>
+      <li></li>
+    </ul>
+  </li -->
+</ul>
+
 <h3 id="ttssh_2.91">2019.12.7 (Ver 2.91)</h3>
 <ul class="history">
   <li>\x95ύX

Modified: branches/4-stable/ttssh2/ttxssh/key.c
===================================================================
--- branches/4-stable/ttssh2/ttxssh/key.c	2020-02-19 17:59:21 UTC (rev 8553)
+++ branches/4-stable/ttssh2/ttxssh/key.c	2020-02-19 18:03:24 UTC (rev 8554)
@@ -239,7 +239,7 @@
 	}
 	decrypted = malloc(rsasize);
 	if (decrypted == NULL)
-		return 1; // error
+		return 0; // error
 
 	if ((len = RSA_public_decrypt(siglen, sigbuf, decrypted, rsa,
 	                              RSA_PKCS1_PADDING)) < 0) {


Ttssh2-commit メーリングリストの案内
Back to archive index