Use the | trick to save a comparison in our calloc check.

This commit is contained in:
Nick Mathewson 2014-11-02 11:48:08 -05:00
parent 0d8abf5365
commit ded33cb2c7

View File

@ -206,8 +206,15 @@ tor_malloc_zero_(size_t size DMALLOC_PARAMS)
static INLINE int
size_mul_check(const size_t x, const size_t y)
{
return ((x < SQRT_SIZE_MAX_P1 && y < SQRT_SIZE_MAX_P1) ||
y == 0 || x <= SIZE_MAX / y);
/* This first check is equivalent to
(x < SQRT_SIZE_MAX_P1 && y < SQRT_SIZE_MAX_P1)
Rationale: if either one of x or y is >= SQRT_SIZE_MAX_P1, then it
will have some bit set in its most significant half.
*/
return ((x|y) < SQRT_SIZE_MAX_P1 ||
y == 0 ||
x <= SIZE_MAX / y);
}
/** Allocate a chunk of <b>nmemb</b>*<b>size</b> bytes of memory, fill