diff options
| author |   Damjan Marion <damarion@cisco.com> | 2016-08-16 11:27:02 +0200 |
|---|---|---|
| committer |   Florin Coras <florin.coras@gmail.com> | 2016-08-17 08:21:42 +0000 |
| commit | 3f54b18e055a606dd0ade78a370eebb758d8be7d (patch) | |
| tree | a1e449d7f324401534130f377c83afb7efee8dea /vnet/vnet/ipsec/esp_decrypt.c | |
| parent | 607de1a0638fa45db49295f9ed51a7f9a5e38706 (diff) | |
Fix coverity issues in IPSec code, fixes VPP-189
Change-Id: I4e43606884ebad9a84abda779b82417192727ef3
Signed-off-by: Damjan Marion <damarion@cisco.com>
Diffstat (limited to 'vnet/vnet/ipsec/esp_decrypt.c')
| -rw-r--r-- | vnet/vnet/ipsec/esp_decrypt.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/vnet/vnet/ipsec/esp_decrypt.c b/vnet/vnet/ipsec/esp_decrypt.c index 49b1dfe4735..c350508917b 100644 --- a/vnet/vnet/ipsec/esp_decrypt.c +++ b/vnet/vnet/ipsec/esp_decrypt.c @@ -484,11 +484,11 @@ esp_decrypt_node_fn (vlib_main_t * vm, { o_b0->flags |= VLIB_BUFFER_IS_TRACED; o_b0->trace_index = i_b0->trace_index; + esp_decrypt_trace_t *tr = + vlib_add_trace (vm, node, o_b0, sizeof (*tr)); + tr->crypto_alg = sa0->crypto_alg; + tr->integ_alg = sa0->integ_alg; } - esp_decrypt_trace_t *tr = - vlib_add_trace (vm, node, o_b0, sizeof (*tr)); - tr->crypto_alg = sa0->crypto_alg; - tr->integ_alg = sa0->integ_alg; } vlib_validate_buffer_enqueue_x1 (vm, node, next_index, to_next, @@ -501,7 +501,8 @@ esp_decrypt_node_fn (vlib_main_t * vm, from_frame->n_vectors); free_buffers_and_exit: - vlib_buffer_free (vm, recycle, vec_len (recycle)); + if (recycle) + vlib_buffer_free (vm, recycle, vec_len (recycle)); vec_free (recycle); return from_frame->n_vectors; } |