net/mlx4_en: Avoid adding steering rules with invalid ring
[ Upstream commit 89c557687a32c294e9d25670a96e9287c09f2d5f ]
Inserting steering rules with illegal ring is an invalid operation,
block it.
Fixes: 820672812f
('net/mlx4_en: Manage flow steering rules with ethtool')
Signed-off-by: Talat Batheesh <talatb@mellanox.com>
Signed-off-by: Tariq Toukan <tariqt@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
cebe1ddda9
commit
ecd9a27e6d
1 changed files with 5 additions and 0 deletions
|
@ -1513,6 +1513,11 @@ static int mlx4_en_flow_replace(struct net_device *dev,
|
||||||
qpn = priv->drop_qp.qpn;
|
qpn = priv->drop_qp.qpn;
|
||||||
else if (cmd->fs.ring_cookie & EN_ETHTOOL_QP_ATTACH) {
|
else if (cmd->fs.ring_cookie & EN_ETHTOOL_QP_ATTACH) {
|
||||||
qpn = cmd->fs.ring_cookie & (EN_ETHTOOL_QP_ATTACH - 1);
|
qpn = cmd->fs.ring_cookie & (EN_ETHTOOL_QP_ATTACH - 1);
|
||||||
|
if (qpn < priv->rss_map.base_qpn ||
|
||||||
|
qpn >= priv->rss_map.base_qpn + priv->rx_ring_num) {
|
||||||
|
en_warn(priv, "rxnfc: QP (0x%x) doesn't exist\n", qpn);
|
||||||
|
return -EINVAL;
|
||||||
|
}
|
||||||
} else {
|
} else {
|
||||||
if (cmd->fs.ring_cookie >= priv->rx_ring_num) {
|
if (cmd->fs.ring_cookie >= priv->rx_ring_num) {
|
||||||
en_warn(priv, "rxnfc: RX ring (%llu) doesn't exist\n",
|
en_warn(priv, "rxnfc: RX ring (%llu) doesn't exist\n",
|
||||||
|
|
Loading…
Add table
Reference in a new issue