-
Type: Bug
-
Resolution: Done
-
Priority: Trivial - P5
-
Affects Version/s: None
-
Component/s: 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.