Bug #3693: visibility hidden functions

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: ruby 1.9.3dev (2010-08-13 trunk 28973) [i686-linux]

[authorNari (Narihiro Nakamura)]

=begin
nari です。

ヘッダファイルに定義されていない関数を何個かみつけました。
修正したパッチを添付します。

定義漏れしていた関数は ext/objspace/objspace.c 内で使われているのですが、
この拡張ライブラリにはテストがなく、気づくのが遅れたようです。

そこで ext/objspace/objspace.c について簡単なテストを書いてみました。
書いたテストを実行した所、数点SEGVする箇所を見つけたので、それについて
も修正しています。

objspaceのメンテナの方からお許し頂ければコミットしたいと思います。
=end

[ko1 (Koichi Sasada)]

=begin
　ささだです．

(2010/08/14 6:56), Narihiro Nakamura wrote:

ヘッダファイルに定義されていない関数を何個かみつけました。
修正したパッチを添付します。

　これらの関数は，まさに objspace.c 用なのと，あまり乱用してほしくないの
で，とりあえずヘッダファイルに入れていません．

そこで ext/objspace/objspace.c について簡単なテストを書いてみました。
書いたテストを実行した所、数点SEGVする箇所を見つけたので、それについて
も修正しています。

　具体的に，SEGV したのはどこで，修正はどれでしょうか．と思った
ら，fixed_objspace_test.rb の中かー．こっちは良いと思います．

--
// SASADA Koichi at atdot dot net
=end

[ko1 (Koichi Sasada)]

=begin
　ささだです．

(2010/08/14 6:56), Narihiro Nakamura wrote:

ヘッダファイルに定義されていない関数を何個かみつけました。
修正したパッチを添付します。

　これらの関数は，まさに objspace.c 用なのと，あまり乱用してほしくないの
で，とりあえずヘッダファイルに入れていません．

そこで ext/objspace/objspace.c について簡単なテストを書いてみました。
書いたテストを実行した所、数点SEGVする箇所を見つけたので、それについて
も修正しています。

　具体的に，SEGV したのはどこで，修正はどれでしょうか．と思った
ら，fixed_objspace_test.rb の中かー．こっちは良いと思います．

--
// SASADA Koichi at atdot dot net

=end

[ko1 (Koichi Sasada)]

=begin
(2010/08/14 7:44), SASADA Koichi wrote:

そこで ext/objspace/objspace.c について簡単なテストを書いてみました。
書いたテストを実行した所、数点SEGVする箇所を見つけたので、それについて
も修正しています。

　具体的に，SEGV したのはどこで，修正はどれでしょうか．と思った
ら，fixed_objspace_test.rb の中かー．こっちは良いと思います．

　あと，このタイミングで 1.9.2 への反映はどうしたものでしょうか．

--
// SASADA Koichi at atdot dot net

=end

[authorNari (Narihiro Nakamura)]

=begin
nari です。

2010年8月14日7:44 SASADA Koichi [e-mail]:

　ささだです．

(2010/08/14 6:56), Narihiro Nakamura wrote:

ヘッダファイルに定義されていない関数を何個かみつけました。
修正したパッチを添付します。

これらの関数は，まさに objspace.c 用なのと，あまり乱用してほしくないの
で，とりあえずヘッダファイルに入れていません．

r28709のコミットで、ヘッダファイルに未定義な関数はC拡張ライブラリから見
えなくなっているようです。
なので、現状としてはヘッダファイルの修正を行わないと objspace は動かな
いと思います。

うーん。推奨しない関数の扱いはどうするといいんでしょうか…。

そこで ext/objspace/objspace.c について簡単なテストを書いてみました。
書いたテストを実行した所、数点SEGVする箇所を見つけたので、それについて
も修正しています。

　具体的に，SEGV したのはどこで，修正はどれでしょうか．と思った
ら，fixed_objspace_test.rb の中かー．こっちは良いと思います．

ありがとうございます。上記の問題が解決したらコミットしたいと思います。

--
Narihiro Nakamura (nari)
=end

[authorNari (Narihiro Nakamura)]

=begin
nari です。

2010年8月14日7:44 SASADA Koichi [e-mail]:

　ささだです．

(2010/08/14 6:56), Narihiro Nakamura wrote:

ヘッダファイルに定義されていない関数を何個かみつけました。
修正したパッチを添付します。

これらの関数は，まさに objspace.c 用なのと，あまり乱用してほしくないの
で，とりあえずヘッダファイルに入れていません．

r28709のコミットで、ヘッダファイルに未定義な関数はC拡張ライブラリから見
えなくなっているようです。
なので、現状としてはヘッダファイルの修正を行わないと objspace は動かな
いと思います。

うーん。推奨しない関数の扱いはどうするといいんでしょうか…。

そこで ext/objspace/objspace.c について簡単なテストを書いてみました。
書いたテストを実行した所、数点SEGVする箇所を見つけたので、それについて
も修正しています。

　具体的に，SEGV したのはどこで，修正はどれでしょうか．と思った
ら，fixed_objspace_test.rb の中かー．こっちは良いと思います．

ありがとうございます。上記の問題が解決したらコミットしたいと思います。

--
Narihiro Nakamura (nari)

=end

[kou (Kouhei Sutou)]

=begin
須藤です。

In [e-mail]
"[ruby-dev:42025] Re: [Bug #3693] visibility hidden functions" on Sat, 14 Aug 2010 08:47:55 +0900,
Narihiro Nakamura [e-mail] wrote:

2010年8月14日7:44 SASADA Koichi [e-mail]:

(2010/08/14 6:56), Narihiro Nakamura wrote:

ヘッダファイルに定義されていない関数を何個かみつけました。
修正したパッチを添付します。

これらの関数は，まさに objspace.c 用なのと，あまり乱用してほしくないの
で，とりあえずヘッダファイルに入れていません．

r28709のコミットで、ヘッダファイルに未定義な関数はC拡張ライブラリから見
えなくなっているようです。
なので、現状としてはヘッダファイルの修正を行わないと objspace は動かな
いと思います。

うーん。推奨しない関数の扱いはどうするといいんでしょうか…。

私は、こういうときは、private.hみたいなインストールされない
ヘッダーファイルを作って、.cの中からだけそれをincludeするよ
うにしています。

=end

[nobu (Nobuyoshi Nakada)]

=begin
なかだです。

At Sat, 14 Aug 2010 09:00:21 +0900,
Kouhei Sutou wrote in [ruby-dev:42026]:

うーん。推奨しない関数の扱いはどうするといいんでしょうか…。

私は、こういうときは、private.hみたいなインストールされない
ヘッダーファイルを作って、.cの中からだけそれをincludeするよ
うにしています。

これも何年か前からのペンディング事項の一つなんですが、なんて名前
にしますかねぇ。includeの下に置かなければインストールされないの
で割とどんなのでもいいんじゃないかとも思いますが。

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

=end

[kou (Kouhei Sutou)]

=begin
須藤です。

In [e-mail]
"[ruby-dev:42028] Re: [Bug #3693] visibility hidden functions" on Sat, 14 Aug 2010 11:10:28 +0900,
Nobuyoshi Nakada [e-mail] wrote:

うーん。推奨しない関数の扱いはどうするといいんでしょうか…。

私は、こういうときは、private.hみたいなインストールされない
ヘッダーファイルを作って、.cの中からだけそれをincludeするよ
うにしています。

これも何年か前からのペンディング事項の一つなんですが、なんて名前
にしますかねぇ。includeの下に置かなければインストールされないの
で割とどんなのでもいいんじゃないかとも思いますが。

今回のやつだとgc.hでもよさそうな気がします。
が、推奨しないだけで使いたい人は（自己責任で）使ってもいいよ、
くらいならinclude/ruby/experimental.hとかに入れて、明示的に
include ruby/experimental.hしないと使えないようにするのでも
よさそうな気がします。

=end

[authorNari (Narihiro Nakamura)]

=begin
nari です。

中田さんの修正のおかげで ext/objspace/objspace.c が動くようになったので
テストとバグ修正をコミットしました(r28986)。

=end

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

=begin

=end

[ko1 (Koichi Sasada)]

=begin
　ささだです．

　SEGV バグなので，1.9.2 にバックポートして頂ければと思います．

(2010/08/14 18:17), Narihiro Nakamura wrote:

チケット #3693 が更新されました。 (by Narihiro Nakamura)

nari です。

中田さんの修正のおかげで ext/objspace/objspace.c が動くようになったので
テストとバグ修正をコミットしました(r28986)。

---

http://redmine.ruby-lang.org/issues/show/3693

---

http://redmine.ruby-lang.org

--
// SASADA Koichi at atdot dot net

=end

[yugui (Yuki Sonoda)]

=begin
On 8/22/10 11:31 PM, SASADA Koichi wrote:

　SEGV バグなので，1.9.2 にバックポートして頂ければと思います．

はい。十分に「枯れた」パッチですし、バックポートします。

--
Yugui [e-mail]
http://yugui.jp
私は私をDumpする

=end