Message ID | 20200522052348.1241-1-wu000273@umn.edu (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | net/mlx4_core: fix a memory leak bug. | expand |
On Fri, 22 May 2020 00:23:48 -0500 wu000273@umn.edu wrote: > From: Qiushi Wu <wu000273@umn.edu> > > In function mlx4_opreq_action(), pointer "mailbox" is not released, > when mlx4_cmd_box() return and error, causing a memory leak bug. > Fix this issue by going to "out" label, mlx4_free_cmd_mailbox() can > free this pointer. > > Fixes: fe6f700d6cbb7 ("Respond to operation request by firmware") > Signed-off-by: Qiushi Wu <wu000273@umn.edu> Fixes tag: Fixes: fe6f700d6cbb7 ("Respond to operation request by firmware") Has these problem(s): - Subject does not match target commit subject Just use git log -1 --format='Fixes: %h ("%s")'
diff --git a/drivers/net/ethernet/mellanox/mlx4/fw.c b/drivers/net/ethernet/mellanox/mlx4/fw.c index 6e501af0e532..f6ff9620a137 100644 --- a/drivers/net/ethernet/mellanox/mlx4/fw.c +++ b/drivers/net/ethernet/mellanox/mlx4/fw.c @@ -2734,7 +2734,7 @@ void mlx4_opreq_action(struct work_struct *work) if (err) { mlx4_err(dev, "Failed to retrieve required operation: %d\n", err); - return; + goto out; } MLX4_GET(modifier, outbox, GET_OP_REQ_MODIFIER_OFFSET); MLX4_GET(token, outbox, GET_OP_REQ_TOKEN_OFFSET);