A failure to allocate a bounce buffer for encap. header
parsing results in falling to the error path but does
not set an appropriate error code. Fix this.
Fixes: 1bbd1ec2348a ("net/sfc: support action VXLAN encap in MAE backend")
Cc: sta...@dpdk.org
Signed-off-by: Ivan Malov <ivan.ma...@arknetworks.am>
Reviewed-by: Andy Moreton <andy.more...@amd.com>
---
.mailmap | 2 +-
drivers/net/sfc/sfc_mae.c | 4 +++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/.mailmap b/.mailmap
index 864d33ee46..ec31ab8bd0 100644
--- a/.mailmap
+++ b/.mailmap
@@ -106,7 +106,7 @@ Andriy Berestovskyy <a...@semihalf.com>
<andriy.berestovs...@caviumnetworks.com>
Andrzej Ostruszka <a...@semihalf.com> <aostrus...@marvell.com>
Andy Gospodarek <andrew.gospoda...@broadcom.com> <go...@broadcom.com>
Andy Green <a...@warmcat.com>
-Andy Moreton <amore...@xilinx.com> <amore...@solarflare.com>
+Andy Moreton <andy.more...@amd.com> <amore...@xilinx.com>
<amore...@solarflare.com>
Andy Pei <andy....@intel.com>
Anirudh Venkataramanan <anirudh.venkatarama...@intel.com>
Ankur Dwivedi <adwiv...@marvell.com> <ankur.dwiv...@caviumnetworks.com>
<ankur.dwiv...@cavium.com>
diff --git a/drivers/net/sfc/sfc_mae.c b/drivers/net/sfc/sfc_mae.c
index f5fe55b46f..bf1c2f60c2 100644
--- a/drivers/net/sfc/sfc_mae.c
+++ b/drivers/net/sfc/sfc_mae.c
@@ -215,8 +215,10 @@ sfc_mae_attach(struct sfc_adapter *sa)
bounce_eh->buf_size = limits.eml_encap_header_size_limit;
bounce_eh->buf = rte_malloc("sfc_mae_bounce_eh",
bounce_eh->buf_size, 0);
- if (bounce_eh->buf == NULL)
+ if (bounce_eh->buf == NULL) {
+ rc = ENOMEM;
goto fail_mae_alloc_bounce_eh;
+ }
mae->nb_outer_rule_prios_max = limits.eml_max_n_outer_prios;
mae->nb_action_rule_prios_max = limits.eml_max_n_action_prios;
--
2.17.1
