| From c2d4c543f74c90f883e8ec62a31973ae8807d354 Mon Sep 17 00:00:00 2001 |
| From: Rakesh Babu Saladi <rsaladi2@marvell.com> |
| Date: Wed, 27 Oct 2021 23:02:34 +0530 |
| Subject: octeontx2-af: Fix possible null pointer dereference. |
| |
| From: Rakesh Babu Saladi <rsaladi2@marvell.com> |
| |
| commit c2d4c543f74c90f883e8ec62a31973ae8807d354 upstream. |
| |
| This patch fixes possible null pointer dereference in files |
| "rvu_debugfs.c" and "rvu_nix.c" |
| |
| Fixes: 8756828a8148 ("octeontx2-af: Add NPA aura and pool contexts to debugfs") |
| Fixes: 9a946def264d ("octeontx2-af: Modify nix_vtag_cfg mailbox to support TX VTAG entries") |
| Signed-off-by: Rakesh Babu Saladi <rsaladi2@marvell.com> |
| Signed-off-by: Subbaraya Sundeep <sbhatta@marvell.com> |
| Signed-off-by: Sunil Goutham <sgoutham@marvell.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c | 2 +- |
| drivers/net/ethernet/marvell/octeontx2/af/rvu_nix.c | 3 +++ |
| 2 files changed, 4 insertions(+), 1 deletion(-) |
| |
| --- a/drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c |
| +++ b/drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c |
| @@ -581,7 +581,7 @@ static ssize_t rvu_dbg_qsize_write(struc |
| if (cmd_buf) |
| ret = -EINVAL; |
| |
| - if (!strncmp(subtoken, "help", 4) || ret < 0) { |
| + if (ret < 0 || !strncmp(subtoken, "help", 4)) { |
| dev_info(rvu->dev, "Use echo <%s-lf > qsize\n", blk_string); |
| goto qsize_write_done; |
| } |
| --- a/drivers/net/ethernet/marvell/octeontx2/af/rvu_nix.c |
| +++ b/drivers/net/ethernet/marvell/octeontx2/af/rvu_nix.c |
| @@ -2146,6 +2146,9 @@ static void nix_free_tx_vtag_entries(str |
| return; |
| |
| nix_hw = get_nix_hw(rvu->hw, blkaddr); |
| + if (!nix_hw) |
| + return; |
| + |
| vlan = &nix_hw->txvlan; |
| |
| mutex_lock(&vlan->rsrc_lock); |