Commit 05e4ecd1 authored by Eli Cohen's avatar Eli Cohen Committed by David S. Miller

net/mlx5_core: Avoid copying outbox in aysnc command completion

Avoid copying to the output buffer in cmd_exec since this is done after the
command is completed. Failure to do this may cause cases where the callback
handler is called before the copy done by cmd_exec which then overwrites it.
Reported-by: default avatarTamer Hleihel <tamerh@mellanox.com>
Signed-off-by: default avatarEli Cohen <eli@mellanox.com>
Signed-off-by: default avatarSaeed Mahameed <saeedm@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 64599cca
...@@ -1236,6 +1236,7 @@ static int cmd_exec(struct mlx5_core_dev *dev, void *in, int in_size, void *out, ...@@ -1236,6 +1236,7 @@ static int cmd_exec(struct mlx5_core_dev *dev, void *in, int in_size, void *out,
goto out_out; goto out_out;
} }
if (!callback)
err = mlx5_copy_from_msg(out, outb, out_size); err = mlx5_copy_from_msg(out, outb, out_size);
out_out: out_out:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment