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

Actions

Backport #7903

closed

Status:

Closed

Assignee:

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

Also available in: Atom PDF

Like0

Like0Like0Like0Like0