diff options
author | Anca Jeanina Floarea <anca.floarea@freescale.com> | 2014-09-15 13:34:40 (GMT) |
---|---|---|
committer | Matthew Weigel <Matthew.Weigel@freescale.com> | 2014-12-11 18:39:31 (GMT) |
commit | ae7062d6481621cc672a0c6a5167a78f8f302cd9 (patch) | |
tree | d9eb23c487f9314cd800f7f60f0d2995f73fdaff /drivers/staging | |
parent | 39585ee7882d39c17fd764424a427ea5a96da3c7 (diff) | |
download | linux-fsl-qoriq-ae7062d6481621cc672a0c6a5167a78f8f302cd9.tar.xz |
dpa_offload: Check SA returned by get_sa_from_sa_id
In function dpa_ipsec_free the SA returned by function
get_sa_from_sa_id can be NULL in case of an error. Fixed
this by adding BUG_ON on returned SA.
Signed-off-by: Anca Jeanina Floarea <anca.floarea@freescale.com>
Change-Id: Id663dec2c47cd2eacce8c2a5ca1f00639df08bba
Reviewed-on: http://git.am.freescale.net:8181/18802
Reviewed-by: Marian-Cornel Chereji <marian.chereji@freescale.com>
Tested-by: Marian-Cornel Chereji <marian.chereji@freescale.com>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/fsl_dpa_offload/dpa_ipsec.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/staging/fsl_dpa_offload/dpa_ipsec.c b/drivers/staging/fsl_dpa_offload/dpa_ipsec.c index 3e14ca5..0c0c311 100644 --- a/drivers/staging/fsl_dpa_offload/dpa_ipsec.c +++ b/drivers/staging/fsl_dpa_offload/dpa_ipsec.c @@ -3657,6 +3657,7 @@ int dpa_ipsec_free(int dpa_ipsec_id) sa_id = instance->used_sa_ids[i]; if (sa_id != DPA_OFFLD_INVALID_OBJECT_ID) { sa = get_sa_from_sa_id(instance, sa_id); + BUG_ON(!sa); if (sa_is_inbound(sa)) { if (sa_is_child(sa)) remove_inbound_sa(sa->parent_sa); |