diff options
| author |   Benoît Ganne <bganne@cisco.com> | 2021-11-19 13:39:10 +0100 |
|---|---|---|
| committer |   Damjan Marion <dmarion@me.com> | 2021-11-22 10:06:48 +0000 |
| commit | 563ec99fa371c473da711483d25517052e9867b9 (patch) | |
| tree | 0ee34a5a68c71bd95f3dea3f62e88443676106e8 | |
| parent | 3d5e48ba2499fe6a01f48f43a057f3d985ec0ff5 (diff) | |
vppinfra: annotate pool_dup() for asan
When using AddressSanitizer, pool objects are poisoned, but pool_dup()
deep copy is ok.
Type: fix
Change-Id: Id801c4ab35666474a297d885c4b92878e44feab6
Signed-off-by: Benoît Ganne <bganne@cisco.com>
| -rw-r--r-- | src/vppinfra/pool.h | 46 |
1 files changed, 24 insertions, 22 deletions
diff --git a/src/vppinfra/pool.h b/src/vppinfra/pool.h index 6f16e617cd2..68a859edc8f 100644 --- a/src/vppinfra/pool.h +++ b/src/vppinfra/pool.h @@ -373,28 +373,30 @@ do { \ * @param A alignment (may be zero) * @return copy of pool */ -#define pool_dup_aligned(P,A) \ -({ \ - typeof (P) _pool_var (new) = 0; \ - pool_header_t * _pool_var (ph), * _pool_var (new_ph); \ - u32 _pool_var (n) = pool_len (P); \ - if ((P)) \ - { \ - _pool_var (new) = _vec_resize (_pool_var (new), _pool_var (n), \ - _pool_var (n) * sizeof ((P)[0]), \ - pool_aligned_header_bytes, (A)); \ - clib_memcpy_fast (_pool_var (new), (P), \ - _pool_var (n) * sizeof ((P)[0])); \ - _pool_var (ph) = pool_header (P); \ - _pool_var (new_ph) = pool_header (_pool_var (new)); \ - _pool_var (new_ph)->free_bitmap = \ - clib_bitmap_dup (_pool_var (ph)->free_bitmap); \ - _pool_var (new_ph)->free_indices = \ - vec_dup (_pool_var (ph)->free_indices); \ - _pool_var (new_ph)->max_elts = _pool_var (ph)->max_elts; \ - } \ - _pool_var (new); \ -}) +#define pool_dup_aligned(P, A) \ + ({ \ + typeof (P) _pool_var (new) = 0; \ + pool_header_t *_pool_var (ph), *_pool_var (new_ph); \ + u32 _pool_var (n) = pool_len (P); \ + if ((P)) \ + { \ + _pool_var (new) = _vec_resize (_pool_var (new), _pool_var (n), \ + _pool_var (n) * sizeof ((P)[0]), \ + pool_aligned_header_bytes, (A)); \ + CLIB_MEM_OVERFLOW_PUSH ((P), _pool_var (n) * sizeof ((P)[0])); \ + clib_memcpy_fast (_pool_var (new), (P), \ + _pool_var (n) * sizeof ((P)[0])); \ + CLIB_MEM_OVERFLOW_POP (); \ + _pool_var (ph) = pool_header (P); \ + _pool_var (new_ph) = pool_header (_pool_var (new)); \ + _pool_var (new_ph)->free_bitmap = \ + clib_bitmap_dup (_pool_var (ph)->free_bitmap); \ + _pool_var (new_ph)->free_indices = \ + vec_dup (_pool_var (ph)->free_indices); \ + _pool_var (new_ph)->max_elts = _pool_var (ph)->max_elts; \ + } \ + _pool_var (new); \ + }) /** * Return copy of pool without alignment |