diff options
| author |   Alberto Compagno <acompagn+fdio@cisco.com> | 2019-12-13 20:43:30 +0000 |
|---|---|---|
| committer | Alberto Compagno <acompagn+fdio@cisco.com> | 2019-12-13 20:53:47 +0000 |
| commit | 2cf6ae9b832f3709209a63778e85ca46d3e1a1ec (patch) | |
| tree | e8152d33fa164538046a033daa355fef32ad7b22 /hicn-plugin/src/strategy.c | |
| parent | 02be1b9403b12c03fc4c03ef872fb71ec37b2c1c (diff) | |
[HICN-457] Adding missing lock to the hash entry in order to avoid memory leak.
Signed-off-by: Alberto Compagno <acompagn+fdio@cisco.com>
Change-Id: I9ee6d6311ca39353f93e6d41b06752b17628b46f
Diffstat (limited to 'hicn-plugin/src/strategy.c')
| -rw-r--r-- | hicn-plugin/src/strategy.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/hicn-plugin/src/strategy.c b/hicn-plugin/src/strategy.c index 62c2ddc8b..15c0dc720 100644 --- a/hicn-plugin/src/strategy.c +++ b/hicn-plugin/src/strategy.c @@ -117,6 +117,9 @@ hicn_new_interest (hicn_strategy_runtime_t * rt, vlib_buffer_t * b0, hicn_store_internal_state (b0, hicnb0->name_hash, node_id0, dpo_ctx_id0, vft_id0, hash_entry_id, bucket_id, bucket_is_overflow); + // We need to take a lock as the lock is not taken on the hash + // entry because it is a CS entry (hash_insert function). + hash_entry->locks++; *next = is_cs0 ? HICN_STRATEGY_NEXT_INTEREST_HITCS : HICN_STRATEGY_NEXT_INTEREST_HITPIT; |