[Scim-imengine-dev 307] Re: Scim-anthyのsegfaultに対するパッチ

アーカイブの一覧に戻る

Takuro Ashie ashie****@homa*****
2005年 5月 9日 (月) 12:49:21 JST


足永です.

返事が遅くなってしまい申し訳ありません.
自宅の環境を再構築するのに手間取ってました.

# 某社のルータはもう絶対買わない...

On Thu, 05 May 2005 13:24:37 +0900
大力亮 <ryo-d****@mbm*****> wrote:

> 1. substrの第二引数がおかしい。
> String#substrの第二引数は取得部分の長さ指定ですが、一部で取得部分の最終
> 位置指定と勘違いして
> 使っています。これらが特定の操作でsegfaultを起こすのを確認したため修正し
> ました。
> 2. erase()の実装がまずい。
> erase()内部でappend_str()を呼ぶ場合がありますが、これも特定操作で
> segfaultを起こします。

私の汚いコードを解読して下さってありがとうございます(-人-

@@ -397,6 +394,11 @@
         m_char_list.erase(begin, end);
     }
 
+    if (m_key2kana.is_pending ()) {
+        m_key2kana.clear ();
+    }

をreset_pending()に移動して取り込み,0.3.2としてリリースしようと思います.
このパッチで,内海さんが指摘して下さった[Scim-imengine-dev 272]の問題も直るようです.

2.の問題は,erase()するlengthを指定できるようにしていた頃の名残りという
か修正漏れのようです.最終的には,面倒なので長さは一文字固定で妥協しました.


> 元の機能を損なわない様、修正したつもりですが、もしかすると一部の機能が失
> われた可能性もあります。

問題無いと思います.


> 追伸:
> もしよろしければ、SCIM IMEngineのパッケージャーをやらせていただけませんか。
> 一部のパッケージは既に、出来上がっているのですが、アップロードする場所が
> 無くて困っています。

是非是非お願いします.
sf.jpのアカウントを教えて下さい.
メンバーに追加します.



Scim-imengine-dev メーリングリストの案内
アーカイブの一覧に戻る