-
Type:
Bug
-
Resolution: Done
-
Priority:
Trivial - P5
-
Affects Version/s: None
-
Component/s: None
-
None
-
None
-
None
alexander.gorrod, there's an oddity in test_bloom.c:
/*
* Try out some values we didn't insert - choose a different size to
* ensure the value doesn't overlap with existing values.
*/
item.size = g.c_key_max + 10;
item.data = calloc(item.size, 1);
memset((void *)item.data, 'a', item.size);
for (i = 0, fp = 0; i < g.c_ops; i++) {
((uint8_t *)item.data)[i % item.size] =
'a' + ((uint8_t)rand() % 26);
if ((ret = __wt_bloom_get(bloomp, &item)) == 0)
++fp;
}
A value is assigned to ret, but it's never used. I'm not sure what's going on here, so I'm punting it to you.
There's also a calloc immediately above that isn't checked for failure.