プロジェクト

全般

プロフィール

バグ #951

詳細タブのツイート時刻をクリックするとmikutterが暴走する.

Shi rusu10ヶ月前に追加. 10ヶ月前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
開始日:
2016-12-20
プラグイン名:

説明

ツイートを開いたときに表示される 詳細タブ 中の ツイート時刻 をクリックすると, mikutterが暴走します.
関連付けをすべて消した状態でもそうなります.

log.txt (76.8 KB) log.txt あひる 家鴨, 2016-12-20 14:58
0001-display_requirements-Add-guard-to-be-message-id-unde.patch (1.32 KB) 0001-display_requirements-Add-guard-to-be-message-id-unde.patch コスモ 虹川, 2016-12-23 00:21

関係しているリビジョン

リビジョン 1c15113d (差分)
toshi_a 初音10ヶ月前に追加

message_detail_view: 日付をクリックした時にIntentToken#forward を呼ぶ refs #951

履歴

#1 toshi_a 初音10ヶ月前に更新

暴走って具体的にどんな感じですか?フリーズして動かなくなるとか?

#2 Shi rusu10ヶ月前に更新

大量にCPUリソースを消費したまま, 一切操作を受け付けなくなります.

#3 toshi_a 初音10ヶ月前に更新

  • トラッカー機能 から バグ に変更

なるほど、使える状態ではなさそうですね

#4 あひる 家鴨10ヶ月前に更新

developブランチですが、同様の手順でmikutterが落ちたので取れたログを添付します
現象としてはセグって落ちてるように見えます

#5 Akira Ouchi10ヶ月前に更新

私の環境で確認した場合は以下のエラーとともにクラッシュしました。

/Users/akkie/mikutter_dev/core/mui/gtk_retriever_header_widget.rb:71:in `block in message_opener': undefined local variable or method `retriever' for #<Gtk::RetrieverHeaderWidget:0x7ffb1a310b88 ptr=0x7ffb14e06400>
     from /Users/akkie/mikutter_dev/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /Users/akkie/mikutter_dev/core/plugin/gtk/mainloop.rb:10:in `main'
     from /Users/akkie/mikutter_dev/core/plugin/gtk/mainloop.rb:10:in `mainloop'
     from mikutter.rb:66:in `boot!'
     from mikutter.rb:95:in `<main>'
notice: mikutter.rb:72:in `rescue in boot!': catch exception `SystemExit'
notice: mikutter.rb:74:in `rescue in boot!': => `SystemExit'
notice: mikutter.rb:99:in `rescue in <main>': catch SystemExit
notice: {MIKUTTER_DIR}/core/configloader.rb:32:in `block in <module:ConfigLoader>': configloader: wrote 1 keys (UserConfig::ui_tab_order)

release/3.5ブランチのcommit fd560042e1d698be888363e16fe3380550d844c4です。

#6 コスモ 虹川10ヶ月前に更新

dd1ac9aff50d0f667387ee0af12d2e26d6657c43 では通常のツイートでは開く方法が提示されるダイアログがでることになっており、
落ちなくなっているようです。

ただし、システムメッセージではまだ落ちるようです。

/media/Data2/mikutter/core/plugin/display_requirements/display_requirements.rb:181:in `clicked': undefined method `id' for #<Mikutter::System::Message:0x0055fa2bfcc530>
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:160:in `block in event_hooks'
     from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `signal_emit'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `block in tree='
     from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `main'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `mainloop'
     from mikutter.rb:66:in `boot!'
     from mikutter.rb:95:in `<main>'
/media/Data2/mikutter/vendor/bundle/ruby/2.3.0/gems/glib2-3.0.8/lib/glib2.rb:41:in `exit': exit
     from /media/Data2/mikutter/vendor/bundle/ruby/2.3.0/gems/glib2-3.0.8/lib/glib2.rb:41:in `exit_application'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `signal_emit'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `block in tree='
     from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `main'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `mainloop'
     from mikutter.rb:66:in `boot!'
     from mikutter.rb:95:in `<main>'
notice: mikutter.rb:72:in `rescue in boot!': catch exception `SystemExit'
notice: mikutter.rb:74:in `rescue in boot!': => `SystemExit'
notice: mikutter.rb:99:in `rescue in <main>': catch SystemExit
notice: {MIKUTTER_DIR}/core/configloader.rb:32:in `block in <module:ConfigLoader>': configloader: wrote 1 keys (UserConfig::ui_tab_order)

修正がアドホックですがこれで一応システムメッセージの場合も落ちなくなります。

#7 toshi_a 初音10ヶ月前に更新

パッチ見ましたが、コスモさんが言ってるのはツイートの詳細じゃなくてMiraclePainterですかね。とすると別の問題?あとで検証します

#8 toshi_a 初音10ヶ月前に更新

  • ステータス新規 から レビュー待ち に変更
  • 担当者Shi rusu にセット
  • 進捗率0 から 100 に変更

報告されたバグも確かにあるんですが、ここの正しい挙動は他のIntentを開く、vanillaのmikutterでの動作でいうと単にブラウザでそのツイートを開くという挙動が正しいので、実装しました。release/3.5ブランチです。
一応そちらでも確認してもらえますか。

日も迫ってるので、 @Akkiesoft も確認してみてもらって、どちらかから報告貰えたらこれでOKとします。

#9 Shi rusu10ヶ月前に更新

  • ステータスレビュー待ち から 解決 に変更

私の環境で試したところ正常に動作するようになりました.

#10 Akira Ouchi10ヶ月前に更新

ブラウザが開けることを確認できました

#11 toshi_a 初音10ヶ月前に更新

  • ステータス解決 から 終了 に変更

他の形式にエクスポート: Atom PDF