diff options
author | Jieqiang Wang <jieqiang.wang@arm.com> | 2021-07-29 17:03:16 +0000 |
---|---|---|
committer | Dave Barach <dave@barachs.net> | 2021-10-04 10:30:54 -0400 |
commit | 039f289e516b073f6db67f7b35aa3aa64fdc9c82 (patch) | |
tree | 0cf73798f3641af66b35b463ac6945e8789e364c /src/vppinfra/test_pool_iterate.c | |
parent | 2c0dc3e58625062da2018ff7aa15c8a0b2dbbc3c (diff) |
vppinfra: fix potential memory access error in _pool_init_fixed
_pool_init_fixed uses mmap to initialize a fixed-size and preallocated
pool, whose size is the sum of vector_size and free_index_size with
alignment to the CLIB_CACHE_LINE_BYTES and page size. In this way
vector_size equals to pool_header_t + vec_header_t + elt_size * max_elts
so moving to the end of the pool space should be pool_header_t pointer +
vector_size, instead of vec_header_t pointer + vector_size.
Simple code to reproduce this error:
u64 *pool;
pool_init_fixed(pool, 2042);
Improve unit test to cover this case
Type: fix
Signed-off-by: Jieqiang Wang <jieqiang.wang@arm.com>
Reviewed-by: Lijian Zhang <lijian.zhang@arm.com>
Reviewed-by: Tianyu Li <tianyu.li@arm.com>
Change-Id: If088ef89b3dcb2d874ee837ae9da60983b14615c
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'src/vppinfra/test_pool_iterate.c')
0 files changed, 0 insertions, 0 deletions