summaryrefslogtreecommitdiff
path: root/proto/bgp/attrs.c
diff options
context:
space:
mode:
authorToke Høiland-Jørgensen <toke@toke.dk>2020-11-24 02:32:13 +0100
committerOndrej Zajicek (work) <santiago@crfreenet.org>2020-11-24 02:36:31 +0100
commitdb2d29073acfd4086fca18ba43a5ed6baccaa8ad (patch)
tree59901d070b0c1c1ba6a1e42fc4957898c15820d6 /proto/bgp/attrs.c
parent3347aaafec5b269637604d1ea5f5969797beadea (diff)
lib/slab: introduce sl_allocz() function and use it in Babel
The babel protocol code was initialising objects returned from the slab allocator by assigning to each of the struct members individually, but wasn't touching the NODE member while doing so. This leads to warnings on debug builds since commit: baac7009063d ("List expensive check.") To fix this, introduce an sl_allocz() variant of the slab allocator which will zero out the memory before returning it, and switch all the babel call sites to use this version. The overhead for doing this should be negligible for small objects, and in the case of babel, the largest object being allocated was being zeroed anyway, so we can drop the memset in babel_read_tlv().
Diffstat (limited to 'proto/bgp/attrs.c')
0 files changed, 0 insertions, 0 deletions