ruby-gnome2-devel-ja

[ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV

[KATO K. <kz...@ro...>]

こんにちは。

Gtk::TextBuffer#insert の文字列を渡すべき部分に nil を渡すと
  TextView_insert.rb:8: [BUG] Segmentation fault
となって SEGV で落ちます。

もちろん、使いかたが間違っているんですが、SEGV で落ちるよりは例外を投
げるべきじゃないでしょうか？

以下のコードで再現します。

require 'gtk2'

Gtk.init
window = Gtk::Window.new

text_view = Gtk::TextView.new
iter = text_view.buffer.get_iter_at_offset(0)
text_view.buffer.insert(0, nil) # SEGV!

window.add(button)
window.show_all

Gtk.main

--
KATO Kazuyoshi  +++[>+++++[>+++++<-]<-]>>.----------.>+++++[<+
++++>-]<.-----.++++.----------.++++.-----------.+.>++++++++++.

Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV

[Nobuyoshi N. <nob...@so...>]

なかだです。

At Wed, 16 Jul 2003 18:08:58 +0900,
KATO Kazuyoshi wrote:
> Gtk::TextBuffer#insert の文字列を渡すべき部分に nil を渡すと
>   TextView_insert.rb:8: [BUG] Segmentation fault
> となって SEGV で落ちます。
> 
> もちろん、使いかたが間違っているんですが、SEGV で落ちるよりは例外を投
> げるべきじゃないでしょうか？

RVAL2CSTR(=StringValuePtr)とRSTRING()->lenを同じ式で使うのは、
評価順序が未定義なのでまずいでしょうね。

-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦

Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV

[Masahiro S. () <sa...@to...>]

さかいです。

From: Nobuyoshi Nakada <nob...@so...>
Subject: Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV
Date: Wed, 16 Jul 2003 19:20:41 +0900

> なかだです。
> 
> At Wed, 16 Jul 2003 18:08:58 +0900,
> KATO Kazuyoshi wrote:
> > Gtk::TextBuffer#insert の文字列を渡すべき部分に nil を渡すと
> >   TextView_insert.rb:8: [BUG] Segmentation fault
> > となって SEGV で落ちます。
> > 
> > もちろん、使いかたが間違っているんですが、SEGV で落ちるよりは例外を投
> > げるべきじゃないでしょうか？
> 
> RVAL2CSTR(=StringValuePtr)とRSTRING()->lenを同じ式で使うのは、
> 評価順序が未定義なのでまずいでしょうね。

うわぁ、確かに。

しかも他にも同じ事をしてる箇所が結構あるなぁ……

--
酒井 政裕 / Masahiro Sakai

Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV

[Nobuyoshi N. <nob...@so...>]

なかだです。

At Thu, 17 Jul 2003 02:09:03 +0900,
Masao Mutoh wrote:
> rbglib_convert.cみたいに StringValue(str)してから、RSTRING(str)->ptr/len
> を使えば良い？

それでいいはずです。

-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦

Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV

[Masahiro S. () <sa...@to...>]

さかいです。

From: Nobuyoshi Nakada <nob...@so...>
Subject: Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV
Date: Thu, 17 Jul 2003 02:31:17 +0900

> なかだです。
> 
> At Thu, 17 Jul 2003 02:09:03 +0900,
> Masao Mutoh wrote:
> > rbglib_convert.cみたいに StringValue(str)してから、RSTRING(str)->ptr/len
> > を使えば良い？
> 
> それでいいはずです。

気がついた所は直しておきました。

--
酒井 政裕 / Masahiro Sakai

Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV

[Masao M. <mu...@hi...>]

むとうです。

On Fri, 18 Jul 2003 22:46:56 +0900 (JST)
Masahiro Sakai (酒井政裕) <sa...@to...> wrote:

> さかいです。
> 
> From: Nobuyoshi Nakada <nob...@so...>
> Subject: Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV
> Date: Thu, 17 Jul 2003 02:31:17 +0900
> 
> > なかだです。
> > 
> > At Thu, 17 Jul 2003 02:09:03 +0900,
> > Masao Mutoh wrote:
> > > rbglib_convert.cみたいに StringValue(str)してから、RSTRING(str)->ptr/len
> > > を使えば良い？
> > 
> > それでいいはずです。
> 
> 気がついた所は直しておきました。

ありがとうございます。助かります。

ついでといっては何ですが、signalsとpropertiesのデフォルトを
instance_methodsの仕様に合わせて、superclass
まで全部読み込むように仕様変更するってやつやりませんか？
＃やるやると言ってて全然手をつけてません....。

-- 
.:% Masao Mutoh<mu...@hi...>

Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV

[Masao M. <mu...@hi...>]

むとうです。

On Wed, 16 Jul 2003 22:15:11 +0900 (JST)
Masahiro Sakai (酒井政裕) <sa...@to...> wrote:

> さかいです。
> 
> From: Nobuyoshi Nakada <nob...@so...>
> Subject: Re: [ruby-gnome2-devel-ja] Gtk::TextBuffer#insert SEGV
> Date: Wed, 16 Jul 2003 19:20:41 +0900
> 
> > なかだです。
> > 
> > At Wed, 16 Jul 2003 18:08:58 +0900,
> > KATO Kazuyoshi wrote:
> > > Gtk::TextBuffer#insert の文字列を渡すべき部分に nil を渡すと
> > >   TextView_insert.rb:8: [BUG] Segmentation fault
> > > となって SEGV で落ちます。
> > > 
> > > もちろん、使いかたが間違っているんですが、SEGV で落ちるよりは例外を投
> > > げるべきじゃないでしょうか？
> > 
> > RVAL2CSTR(=StringValuePtr)とRSTRING()->lenを同じ式で使うのは、
> > 評価順序が未定義なのでまずいでしょうね。
> 
> うわぁ、確かに。
> 
> しかも他にも同じ事をしてる箇所が結構あるなぁ……

すみませんすみません。全部私です(T_T)。

で、これってどうすれば良いんですか？(^^;)

rbglib_convert.cみたいに StringValue(str)してから、RSTRING(str)->ptr/len
を使えば良い？

＃今日は寝ます....。 

-- 
.:% Masao Mutoh<mu...@hi...>