net: microchip: sparx5: Fix uninitialized variable in vcap_path_exist()
authorDan Carpenter <error27@gmail.com>
Fri, 20 Jan 2023 13:47:12 +0000 (16:47 +0300)
committerJakub Kicinski <kuba@kernel.org>
Tue, 24 Jan 2023 05:34:59 +0000 (21:34 -0800)
The "eport" variable needs to be initialized to NULL for this code to
work.

Fixes: 814e7693207f ("net: microchip: vcap api: Add a storage state to a VCAP rule")
Signed-off-by: Dan Carpenter <error27@gmail.com>
Reviewed-by: Steen Hegelund <Steen.Hegelund@microchip.com>
Link: https://lore.kernel.org/r/Y8qbYAb+YSXo1DgR@kili
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/microchip/vcap/vcap_api.c

index d9cf2cd1925a06e7c1b46231044994ed0ace7a9d..2eaa857d8c1af14b3443c52a46a5e95cb3cf6608 100644 (file)
@@ -2010,7 +2010,8 @@ static int vcap_get_next_chain(struct vcap_control *vctrl,
 static bool vcap_path_exist(struct vcap_control *vctrl, struct net_device *ndev,
                            int dst_cid)
 {
-       struct vcap_enabled_port *eport, *elem;
+       struct vcap_enabled_port *eport = NULL;
+       struct vcap_enabled_port *elem;
        struct vcap_admin *admin;
        int tmp;