diff options
| author | Donald Sharp <sharpd@nvidia.com> | 2020-11-14 15:32:49 -0500 |
|---|---|---|
| committer | Donald Sharp <sharpd@nvidia.com> | 2020-11-14 15:32:49 -0500 |
| commit | dc52beced1ef553a4f17c63859cbff0f1ea225b7 (patch) | |
| tree | e8c845b5d0d0074aaa2ec38994c610abafff09a1 /lib/command.c | |
| parent | 15675d025dc024733a532e1a9ad19c59df4ba73d (diff) | |
bgpd: Fix missed unlocks
When iterating over the bgp_dest table, using this pattern:
for (dest = bgp_table_top(table); dest;
dest = bgp_route_next(dest)) {
If the code breaks or returns in the middle we will not have
properly unlocked the node as that bgp_table_top locks the top
dest and bgp_route_next locks the next dest and unlocks the old
dest.
From code inspection I have found a bunch of places that
we either return in the middle of or a break is issued.
Add appropriate unlocks.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to 'lib/command.c')
0 files changed, 0 insertions, 0 deletions
