iavf/iavf_main: actually log ->src mask when talking about it
authorDaniil Tatianin <d-tatianin@yandex-team.ru>
Tue, 20 Dec 2022 06:32:46 +0000 (09:32 +0300)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Tue, 10 Jan 2023 19:52:38 +0000 (11:52 -0800)
This fixes a copy-paste issue where dev_err would log the dst mask even
though it is clearly talking about src.

Found by Linux Verification Center (linuxtesting.org) with the SVACE
static analysis tool.

Fixes: 0075fa0fadd0 ("i40evf: Add support to apply cloud filters")
Signed-off-by: Daniil Tatianin <d-tatianin@yandex-team.ru>
Reviewed-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/iavf/iavf_main.c

index c4e451ef79422a4cd2bf509e22b7b177cc329b33..adc02adef83a2ad180eb9b7fe1390b6ada1c6e07 100644 (file)
@@ -3850,7 +3850,7 @@ static int iavf_parse_cls_flower(struct iavf_adapter *adapter,
                                field_flags |= IAVF_CLOUD_FIELD_IIP;
                        } else {
                                dev_err(&adapter->pdev->dev, "Bad ip src mask 0x%08x\n",
-                                       be32_to_cpu(match.mask->dst));
+                                       be32_to_cpu(match.mask->src));
                                return -EINVAL;
                        }
                }