RDMA/mlx5: Init dest_type when create flow
authorDaria Velikovsky <daria@mellanox.com>
Tue, 7 Jul 2020 11:02:59 +0000 (14:02 +0300)
committerJason Gunthorpe <jgg@nvidia.com>
Thu, 16 Jul 2020 17:11:53 +0000 (14:11 -0300)
When using action drop dest_type was never assigned to any value.  Add
initialization of dest_type to -1 since 0 is valid.

Fixes: f29de9eee782 ("RDMA/mlx5: Add support for drop action in DV steering")
Link: https://lore.kernel.org/r/20200707110259.882276-1-leon@kernel.org
Signed-off-by: Daria Velikovsky <daria@mellanox.com>
Reviewed-by: Maor Gottlieb <maorg@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Jason Gunthorpe <jgg@nvidia.com>
drivers/infiniband/hw/mlx5/fs.c

index 0d8abb7c3cdf6f312eb39a53c32fed2dbe476430..1a7e6226f11ae4b8cf98b9b0592fdfdaaeeeb742 100644 (file)
@@ -1903,7 +1903,7 @@ static int UVERBS_HANDLER(MLX5_IB_METHOD_CREATE_FLOW)(
        struct mlx5_flow_context flow_context = {.flow_tag =
                MLX5_FS_DEFAULT_FLOW_TAG};
        u32 *offset_attr, offset = 0, counter_id = 0;
-       int dest_id, dest_type, inlen, len, ret, i;
+       int dest_id, dest_type = -1, inlen, len, ret, i;
        struct mlx5_ib_flow_handler *flow_handler;
        struct mlx5_ib_flow_matcher *fs_matcher;
        struct ib_uobject **arr_flow_actions;