Bug #5036: time_modify/struct_modifyの例外メッセージがサブクラスの情報を反映しない

• Status: Closed
• Assignee: ktsj (Kazuki Tsujimoto)
• Target version: -
• ruby -v: -

[ktsj (Kazuki Tsujimoto)]

=begin
辻本です。

time_modify/struct_modifyにて生成される例外のメッセージに
クラス名がハードコーディングされているため、
これらがサブクラスのオブジェクトから呼ばれた場合でも
そのことが例外メッセージから読み取れません。

Timeの例:

class Time2 < Time
end
t = Time2.now
$SAFE=4
t.gmtime # => Insecure: can't modify Time (SecurityError)

Structの例:

class Struct2 < Struct
end
s = Struct2.new("S", :m).new
$SAFE=4
s.m=1 # => Insecure: can't modify Struct (SecurityError)

パッチを添付します。
=end

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Assigned
• Assignee set to ktsj (Kazuki Tsujimoto)

入れちゃっていいんじゃないでしょうか。
あとはrb_check_frozen()のようにinsecureチェック用の関数を作るとか。

[ktsj (Kazuki Tsujimoto)]

コミット権を持っていないので、お手数ですがコミッタの方取り込みをお願いします。

[nobu (Nobuyoshi Nakada)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r32569.
Kazuki, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• error.c (rb_check_trusted): new function to check an object is
  trusted.
• struct.c (rb_struct_modify), time.c (time_modify): check by the
  above function to show proper class names. [Bug #5036]

[ktsj (Kazuki Tsujimoto)]

対応ありがとうございました。

なお、このコミット(r32569)でt.rbが混入してしまっています。

[kosaki (Motohiro KOSAKI)]

• ruby -v changed from ruby 1.9.4dev (2011-07-16 trunk 32562) [x86_64-linux] to -

コミット権を持っていないので、お手数ですがコミッタの方取り込みをお願いします。

取ってください。
どうせコードがあってるだろなーって場合は代理コミットするの面倒くさいです

[kosaki (Motohiro KOSAKI)]

コミット権を持っていないので、お手数ですがコミッタの方取り込みをお願いします。

取ってください。
どうせコードがあってるだろなーって場合は代理コミットするの面倒くさいです

[nobu (Nobuyoshi Nakada)]

(11/07/17 21:47), KOSAKI Motohiro wrote:

コミット権を持っていないので、お手数ですがコミッタの方取り込みをお願いします。

取ってください。
どうせコードがあってるだろなーって場合は代理コミットするの面倒くさいです

前田さんに公開鍵を送ってください。

[nobu (Nobuyoshi Nakada)]

(11/07/17 21:47), KOSAKI Motohiro wrote:

コミット権を持っていないので、お手数ですがコミッタの方取り込みをお願いします。

取ってください。
どうせコードがあってるだろなーって場合は代理コミットするの面倒くさいです

前田さんに公開鍵を送ってください。

[ktsj (Kazuki Tsujimoto)]

ありがとうございます。
では、CommitterHowtoのお作法に則って送らせてもらいます。