Alt+F1 (ヘルプ) するとサクラエディタが落ちる

[beru]

「おま環かもしれんけど https://ci.appveyor.com/api/buildjobs/kq0q4vigajo6axqe/artifacts/sakura-build1247-f1122b83-Win32-Release-Exe.zip を解凍して Alt+F1 (ヘルプ) するとサクラエディタが落ちるね...」

リンクされている EXE は Build 1247 を指していました。Build 1240 では落ちませんでした。

これがちょっと関係してるかなーと思いましたが調べていません>「SAKURA Editor / PatchUnicode / #1116 CFileNameManager::GetIniFileName の問題含みの仕様を正します。」

Originally posted by @ds14050 in #620 (comment)

[beru]

f1122b8 で開発環境から起動してみましたが再現しない事を確認しました。Build 1247 では落ちるので何か再現条件が異なるのかもしれません。

[beru]

パスに起因するみたいですね。

[berryzplus]

現状masterから派生したビルド品で再現しないので、
初期設定がおかしくなってる疑いがあるのかな。

[beru]

パスは関係無いかもしれないです。
下記の３つのファイルだけが含まれている状態だと落ちる事が分かりました。

• sakura.exe (Build 1427 のものでもそうでなくても)
• sakura.ini (Build 1427 からコピー)
• sakura.chm (Build 1427 からコピー)

設定が影響しているのかというと、マルチユーザ構成設定ファイル (sakura.exe.ini) を使っても起きるので同じフォルダにある sakura.chm ファイルを見てるのが関係してるんでしょうか？

[beru]

ローカルの sakura.chm ファイルを開くのに失敗してしまっているのかもしれないですね。

[beru]

CHtmlHelp::InitDllImp でアドレス演算子の記述が足りてないのが原因で CHtmlHelp::m_pfnHtmlHelp
が nullptr のままになって、その関数ポインタを shell.cpp の OpenHtmlHelp 関数で使って 😸 ぬるぽ が起きます。

[berryzplus]

まじか・・・言われてみると凡ミス過ぎる...orz
ご迷惑おかけします。

[beru]

いえいえ、自分がちゃんとレビュー出来てませんでした。

ImportTable::proc の型が void* ってなっているので関数ポインタそのまま渡しちゃってもコンパイル時に問題が検出されないですね。じゃあ型を void** にすれば良いのかというと、う～ん…。traits で関数ポインタのアドレスかどうかを静的にチェック出来るかもしれないですが。

[beru]

Build 1240 でも落ちる事を確認しました。

[ds14050]

濡れ衣でしたね。お疲れ様でした。

[beru]

#644 を merge したのでこの issue は close します。