Patch Set 1:

I think some of the static analysis tools are using it for modelling life times. But perhaps we should rename our malloc to something else instead?

I'm not following. free() in this patch is just incorrect as the objects are allocated from bss section -- not malloc.

View Change

1 comment:

• File src/vendorcode/cavium/bdk/libdram/dram-spd.c:

  • Patch Set #1, Line 63: static uint8_t spd_bufs[4 * 256]; /* FIXME(dhendrix): storage for SPD buffers, assume DDR4 */

    This is where the allocation is coming from. Not malloc.

To view, visit change 37940. To unsubscribe, or for help writing mail filters, visit settings.