(deque.c): inline function upper_pow_two only valid for 32 bit size_t about collections-c HOT 5 CLOSED

After looking through the array.c file I notices another thing. When expanding the array, it is checked whether an overflow happens on increasing the capacity:

    size_t new_capacity = ar->capacity * ar->exp_factor;

    /* As long as the capacity is greater that the expansion factor
     * at the point of overflow, this is check is valid. */
    if (new_capacity <= ar->capacity)
        ar->capacity = CC_MAX_ELEMENTS;
    else
        ar->capacity = new_capacity;

But afterwards the variable new_capacity, which might contain an invalid value in case of an overflow, is used for the allocation of the new buffer:

    void **new_buff = ar->mem_alloc(new_capacity * sizeof(void*));

from collections-c.

Hi, I just looked through my open issues and came across this one. Should I open a quick PR to fix both things and to close the issue?

from collections-c.

@flyxi121
It's definitely not by design. It should have been updated when ARCH_64 was added, but apparently it was overlooked. Thanks for spotting it!

from collections-c.

Yep, that's also a bug. It should be:

void **new_buff = ar->mem_alloc(ar->capacity * sizeof(void*));

Again, thanks for eyeballing the code. These things are near impossible to catch with tests. :)

from collections-c.

@fhilgers
Sure, that would be awesome!

from collections-c.