pike.git/ src/ builtin.cmod

2017-06-09

2017-06-09 12:51:34 by Tobias S. Josefowitz <tobij@tobij.de>

• 27c7e3b96b5f746234ed7d1998708e693a2d9be5 (3 lines) (+2/-1) ^{[ Show | Annotate ]}
  Branch: 8.0

Builtin: random(mapping) no longer exposes PIKE_T_FREE to Pike

Off by one error caused random(mapping) to return elements from the
mapping's freelist.

Fixes [LysLysKOM 22108453] reported by Per Cederqvist. Thanks!

2170:    /* Find a random, nonempty bucket */    bucket=my_rand() % md->hashsize;    while(! md->hash[bucket] ) -  if(++bucket > (size_t)md->hashsize) +  if(++bucket >= (size_t)md->hashsize)    bucket=0;       /* Count entries in bucket */