Project

General

Profile

バグ #951

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

Added by Shi rusu almost 3 years ago. Updated almost 3 years ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
ブランチ:
クラッシュする:

Description

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


Files

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

Associated revisions

Revision 1c15113d (diff)
Added by toshi_a 初音 almost 3 years ago

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

History

#1

Updated by toshi_a 初音 almost 3 years ago

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

#2

Updated by Shi rusu almost 3 years ago

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

#3

Updated by toshi_a 初音 almost 3 years ago

  • Tracker changed from 機能 to バグ

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

#4

Updated by あひる 家鴨 almost 3 years ago

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

#5

Updated by Akira Ouchi almost 3 years ago

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

/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

Updated by コスモ 虹川 almost 3 years ago

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

Updated by toshi_a 初音 almost 3 years ago

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

#8

Updated by toshi_a 初音 almost 3 years ago

  • Status changed from 新規 to レビュー待ち
  • Assignee set to Shi rusu
  • % Done changed from 0 to 100

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

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

#9

Updated by Shi rusu almost 3 years ago

  • Status changed from レビュー待ち to 解決

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

#10

Updated by Akira Ouchi almost 3 years ago

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

#11

Updated by toshi_a 初音 almost 3 years ago

  • Status changed from 解決 to 終了

Also available in: Atom PDF