Project

General

Profile

This project is closed and read-only.

Actions
Copy link

Backport #7903

closed

Array#sampleの引数に渡した乱数生成オブジェクトが負の数を返した場合のエラーメッセージの修正

Backport #7903: Array#sampleの引数に渡した乱数生成オブジェクトが負の数を返した場合のエラーメッセージの修正

Added by sho-h (Sho Hashimoto) over 13 years ago. Updated over 13 years ago.

Status:
Closed
Assignee:
nagachika (Tomoyuki Chikanaga)
[ruby-dev:47061]

Description

=begin
Array#sample に渡す乱数生成オブジェクトが負の数を返す場合、必ず「random number too big」というメッセージの RangeError が発生するようです。

gen = proc { -1 }
class << gen
alias rand call
end
[2,4,6,8,10].sample(random: gen) # => test.rb:5:in `sample': random number too big -1 (RangeError)

RAND_UPTO が呼び出している rb_random_ulong_limited の以下の処理がいずれも unsigned long のため、必ず「too big」になるのではないかと思いました。上記の例だと r に -1 を指定、limit に 4 が指定されると想像しています。

unsigned long r = NUM2ULONG(v);
if (r > limit) {
rb_raise(rb_eRangeError, "random number too big %ld", r);
}

rb_random_real 側ではチェックされているため、わかってる事なのかもしれませんが、念のためチケットを発行してみました。

パッチも作りましたが、適用される場合も NUM2LONG(v)してる行は修正していただいた方がよさそうです。
=end

Files

random.c.patch (834 Bytes) random.c.patch sho-h (Sho Hashimoto), 02/21/2013 10:23 PM
Actions
Copy link

Also available in: PDF Atom