バグ #1494: gtk3: Mastodonアカウントのプロフィールを開くとランダムにSEGVする - mikutter - やること

カスタムクエリ

2021除夜のチケット棚卸し
注目のバグ

操作

リンクをコピー

バグ #1494

完了

機能 #887: gtk3

gtk3: Mastodonアカウントのプロフィールを開くとランダムにSEGVする

Shibafu Midorino さんが2年以上前に追加. 2年以上前に更新.

ステータス:

終了

優先度:

通常

担当者:

toshi_a 初音

対象バージョン:

5.0

プラグイン名:

ブランチ:

クラッシュする:

いいえ

説明

Mastodonアカウントのプロフィールタブを開くと、開いた直後や何か操作をしたランダムなタイミングでSEGVします。

[BUG] object allocation during garbage collection phase

とのことなので、gem側の問題がありそうな気がします。

ファイル

一括ダウンロード

log.txt (8.66 KB) log.txt		Shibafu Midorino, 2021-09-12 18:33
mikutter-1494-coredump-backtrace.txt (62.9 KB) mikutter-1494-coredump-backtrace.txt		Shibafu Midorino, 2021-09-22 02:13
Screenshot_20210925_180130.png (220 KB) Screenshot_20210925_180130.png	gtk2/3での表示比較 (e53664346d816fcb0d10d1796d68cddee9281c25)	Shibafu Midorino, 2021-09-25 18:02

関連するチケット

関連しているバグ #1498: gtk3: 新しいMiraclePainterの不具合まとめ

終了

Shibafu Midorino

操作

チケット # 遅延: 日キャンセル

履歴
コメント
プロパティ更新履歴
関係しているリビジョン

操作

リンクをコピー

toshi_a 初音さんが2年以上前に更新

担当者 を toshi_a 初音 にセット

操作

リンクをコピー

Akira Ouchi さんが2年以上前に更新

SEGVしない場合でも、forecast_font_sizeがcore/mui/gtk_extension.rbから削除されているためundefined methodでコケるようです。

/Users/akkie/mikutter-gtk3/core/mui/gtk_intelligent_textview.rb:119:in `block in gen_body': undefined method `forecast_font_size' for #<Pango::FontDescription:0x00007ffc9d52d068 ptr=0x00007ffca0e7cb90 own=true> (NoMethodError)
    from /Users/akkie/mikutter-gtk3/core/mui/gtk_intelligent_textview.rb:115:in `each'
    from /Users/akkie/mikutter-gtk3/core/mui/gtk_intelligent_textview.rb:115:in `each_with_index'
    from /Users/akkie/mikutter-gtk3/core/mui/gtk_intelligent_textview.rb:115:in `gen_body'
    from /Users/akkie/mikutter-gtk3/core/mui/gtk_intelligent_textview.rb:45:in `initialize'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:69:in `new'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:69:in `cell_widget'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:59:in `block (2 levels) in user_field_table'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:56:in `each'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:56:in `each_with_index'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:56:in `block in user_field_table'
    from <internal:kernel>:90:in `tap'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:55:in `user_field_table'
    from /Users/akkie/mikutter-gtk3/plugin/gui/widget.rb:100:in `method_missing'
    from /Users/akkie/mikutter-gtk3/plugin/mastodon_account_viewer/mastodon_account_viewer.rb:43:in `block (2 levels) in <top (required)>'
    from /Users/akkie/mikutter-gtk3/plugin/gui/widget.rb:71:in `instance_exec'
    from /Users/akkie/mikutter-gtk3/plugin/gui/widget.rb:71:in `instance_eval_with_delegate'
    from /Users/akkie/mikutter-gtk3/plugin/modelviewer/modelviewer.rb:46:in `block (3 levels) in <top (required)>'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/filter.rb:34:in `filtering'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:82:in `block (2 levels) in filtering'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:81:in `each'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:81:in `reduce'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:81:in `block in filtering'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:80:in `catch'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:80:in `filtering'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:212:in `block in collect'
    from /Users/akkie/mikutter-gtk3/plugin/modelviewer/modelviewer.rb:63:in `each'
    from /Users/akkie/mikutter-gtk3/plugin/modelviewer/modelviewer.rb:63:in `each'
    from /Users/akkie/mikutter-gtk3/plugin/modelviewer/modelviewer.rb:63:in `sort_by'
    from /Users/akkie/mikutter-gtk3/plugin/modelviewer/modelviewer.rb:63:in `cluster_initialize'
    from /Users/akkie/mikutter-gtk3/plugin/modelviewer/modelviewer.rb:31:in `block (4 levels) in <top (required)>'
    from /Users/akkie/mikutter-gtk3/plugin/gui/widget.rb:71:in `instance_exec'
    from /Users/akkie/mikutter-gtk3/plugin/gui/widget.rb:71:in `instance_eval_with_delegate'
    from /Users/akkie/mikutter-gtk3/plugin/gui/gui.rb:30:in `block (2 levels) in <top (required)>'
    from /Users/akkie/mikutter-gtk3/plugin/modelviewer/modelviewer.rb:24:in `block (3 levels) in <top (required)>'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/listener.rb:25:in `call'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:241:in `block (2 levels) in call_all_listeners'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:240:in `each'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:240:in `block in call_all_listeners'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:239:in `catch'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:239:in `call_all_listeners'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/pluggaloid-1.7.0/lib/pluggaloid/event.rb:62:in `block in call'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/delayer-1.2.1/lib/delayer/procedure.rb:26:in `run'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/delayer-1.2.1/lib/delayer/extend.rb:126:in `run_once_without_pop_reserve'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/delayer-1.2.1/lib/delayer/extend.rb:117:in `run_once'
    from /Users/akkie/.rbenv/versions/3.0.2/lib/ruby/gems/3.0.0/gems/delayer-1.2.1/lib/delayer.rb:38:in `method_missing'
    from /Users/akkie/mikutter-gtk3/plugin/gtk3/mainloop.rb:23:in `block (2 levels) in mainloop'
    from /Users/akkie/mikutter-gtk3/plugin/gtk3/mainloop.rb:12:in `loop'
    from /Users/akkie/mikutter-gtk3/plugin/gtk3/mainloop.rb:12:in `block in mainloop'
    from /Users/akkie/mikutter-gtk3/plugin/gtk3/mainloop.rb:10:in `catch'
    from /Users/akkie/mikutter-gtk3/plugin/gtk3/mainloop.rb:10:in `mainloop'
    from mikutter.rb:79:in `boot!'
    from mikutter.rb:115:in `<main>'

操作

リンクをコピー

toshi_a 初音さんが2年以上前に更新

関連している バグ #1498: gtk3: 新しいMiraclePainterの不具合まとめ を追加

操作

リンクをコピー

Shibafu Midorino さんが2年以上前に更新

ステータス を 分類待ち から 実装待ち に変更

IntelligentTextViewにて、"style-set" signalを購読していることが直接的な原因。

なんでこれが原因になるのかというと、GC中に次のような処理が発生するためである。

マークフェーズで、GObjectのRubyバインディングクラスのC関数 gobj_mark() が呼ばれる
gobj_mark() では、g_object_class_list_properties() と g_object_get_property() を使い、オブジェクトのプロパティを列挙してマークしていく
IntelligentTextViewの親Widgetに対して gobj_mark() が呼ばれた際、列挙中に GtkWidget.style が参照される
gtk_widget_get_style() では、構造体内のstyleに値がセットされていなかった場合はその場で新しいGtkStyleを生成しセットしてから返すようになっている
_gtk_widget_set_style() が呼び出されると、"style-set" signalが放出される
IntelligentTextView#set_events にて "style-set" にconnectしているため、CからRubyへのコールバックが発生する
signalをトリガーとしたRubyへのコールバック処理ではRubyオブジェクトの割り当てがいくつか発生するため、禁忌を犯したこととなりクラッシュする

対策だが、GtkStyle関連は一律deprecatedなのでこれに関する処理は廃止して代替を模索するのが良いと思われる。

操作

リンクをコピー

Shibafu Midorino さんが2年以上前に更新

ファイル mikutter-1494-coredump-backtrace.txt mikutter-1494-coredump-backtrace.txt を追加

log.txtとは別タイミングのものですが、原因特定に用いたGDB バックトレースのコピーを添付しておきます。

操作

リンクをコピー

Shibafu Midorino さんが2年以上前に更新

IntelligentTextViewの "style-set" の購読およびbg_modifierに関する処理のモチベーションは親Widgetから指定された、または親と同じ色の背景色を適切にセットすることのはず。
現在であれば、親と同じ色の背景色は単にCSSで背景を透過させてしまえば十分な結果が得られる。
この方法なら、親のスタイルが変更されたことを検知して自力で更新する必要もなくなる。

"style-set" の購読は削除し、IntelligentTextViewには自身の背景色を透過させるCSSを割り当てるようにした。

操作

リンクをコピー