fix dangling reference in foreach_key_value_pair 82/13982/2
authorDave Barach <dave@barachs.net>
Mon, 6 Aug 2018 12:43:47 +0000 (08:43 -0400)
committerDave Barach <dave@barachs.net>
Mon, 6 Aug 2018 15:28:38 +0000 (11:28 -0400)
commitab955b1b44f3d606cef51a9471a6562a7135ab3e
tree8b14570f9c508304a1183bca161e9185db8f2f58
parent374819dd5873aee53e447ac3a761df036abd9f0a
fix dangling reference in foreach_key_value_pair

When the user deletes the last entry in a bihash bucket, the bihash
infra frees the bucket's backing storage. If this happens under
clib_bihash_foreach_key_value_pair - and the freed bucket happens to
be the bucket being traversed - the resulting dangling reference can
easily make the wheels fall off.

Simple fix: if (bucket-is-now-empty) double-break.

Change-Id: Idc44247a82ed5d0ba548507b4a53d4c8503ba8bb
Signed-off-by: Dave Barach <dave@barachs.net>
(cherry picked from commit ca45ee73d7c49c7f659c5cd690d3403d440e50f9)
src/vppinfra/bihash_template.c