net/sfc: add missing error code indication to MAE init path

[ upstream commit 1f5e6bb633cafc2789ec74e80bf90fa54ef5b788 ] 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: 1bbd1ec ("net/sfc: support action VXLAN encap in MAE backend") Signed-off-by: Ivan Malov <[email protected]> Reviewed-by: Andy Moreton <[email protected]> Acked-by: Ferruh Yigit <[email protected]>
kevintraynor · Oct 27, 2023 · 41fcdc1 · 41fcdc1
1 parent 3937cb5
commit 41fcdc1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/.mailmap b/.mailmap
@@ -103,7 +103,7 @@ Andriy Berestovskyy <[email protected]> <[email protected]>
 Andrzej Ostruszka <[email protected]> <[email protected]>
 Andy Gospodarek <[email protected]> <[email protected]>
 Andy Green <[email protected]>
-Andy Moreton <[email protected]> <[email protected]>
+Andy Moreton <[email protected]> <[email protected]> <[email protected]>
 Andy Pei <[email protected]>
 Anirudh Venkataramanan <[email protected]>
 Ankur Dwivedi <[email protected]> <[email protected]> <[email protected]>

diff --git a/drivers/net/sfc/sfc_mae.c b/drivers/net/sfc/sfc_mae.c
@@ -281,8 +281,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;