もずはっく日記(2014年7月)

2014年7月24日

TSFモードを有効にしていると、テストでメモリリークが検出される、というバグです。

nsTextStoreの実装が良くなく、テストスイートのメモリリークの検出時には正確なリーク情報が出ていなかったので、原因の特定に時間がかかりましたが、その辺を修正してみると簡単な話でした。

原因は、nsTextStore自体のリークで、これによって、そのメンバがリークしているのを検出されていたのでした。

nsTextStoreは参照カウンタまわりを、自前のコードで実装しており、コンストラクタで1がセットされるようになっていました。このため、

nsRefPtr<nsTextStore> textStore = new nsTextStore();

というコードでインスタンスを生成した段階で、参照カウンタが、1つ余分に増えており、これによって永遠に開放されることがなくなってしまっていました。

再発防止のため、また、メモリリークを検出しやすくするために、widget/windows/WinUtils.hにIUnknown派生クラスの実装時に使えるマクロを作成し、これを利用することで、nsISupportsの派生クラスと同様に扱えるように修正しています。