Bug #96156 for Hash-ConsistentHash: Infinite loop in new()

Mon Jun 02 16:41:01 2014 jfesler [...] gigo.com - Ticket created

Subject:

Infinite loop in new()

while( $buckets_per_waight < 5 ){ $self->{mask} = (($self->{mask} << 2) | 3); } If buckets_per_waight < 5 before this code block, it will remain < 5 . This is demonstrable with: my(@targets) = (1..100); my $chash = Hash::ConsistentHash->new( buckets => \@targets, hash_func => \&crc32 );

Mon Jun 02 19:59:58 2014 karavelov [...] mail.bg - Correspondence added

On Mon Jun 02 16:41:01 2014, jfesler wrote: Show quoted text

> while( $buckets_per_waight < 5 ){ > $self->{mask} = (($self->{mask} << 2) | 3); > } > > > > > If buckets_per_waight < 5 before this code block, it will remain < 5 . > > This is demonstrable with: > > > my(@targets) = (1..100); > > my $chash = Hash::ConsistentHash->new( > buckets => \@targets, > hash_func => \&crc32 > ); >

Confirmed. Fixed version is coming in a munite

Mon Jun 02 19:59:59 2014 The RT System itself - Status changed from 'new' to 'open'

Mon Jun 02 21:14:10 2014 karavelov [...] mail.bg - Correspondence added

On Mon Jun 02 19:59:58 2014, KARAVELOV wrote: Show quoted text

> On Mon Jun 02 16:41:01 2014, jfesler wrote:

> > while( $buckets_per_waight < 5 ){ > > $self->{mask} = (($self->{mask} << 2) | 3); > > } > > > > > > > > > > If buckets_per_waight < 5 before this code block, it will remain < 5 . > > > > This is demonstrable with: > > > > > > my(@targets) = (1..100); > > > > my $chash = Hash::ConsistentHash->new( > > buckets => \@targets, > > hash_func => \&crc32 > > ); > >

> > Confirmed. Fixed version is coming in a munite >

Fixed in version 0.07 that is uploaded on CPAN.

Mon Jun 02 21:14:10 2014 karavelov [...] mail.bg - Status changed from 'open' to 'patched'