mbox series

[v4,0/3] vhost: fix fd an memory leaks

Message ID 20201112171029.354593-1-maxime.coquelin@redhat.com (mailing list archive)
Headers
Series vhost: fix fd an memory leaks |

Message

Maxime Coquelin Nov. 12, 2020, 5:10 p.m. UTC 
  This series fixes several leaks in Vhost-user requests
handling.

Thanks to Xuan Ding from Intel for reporting these
issues.

Changes in v4:
- Fix wrong return value in set_mem_table (Xueming)

Changes in v3:
- Remove the right closing of fds. (Chenbo)
Changes in v2:
- Fix typos in commit messages. (Chenbo)
- Remove useless closing of fds in set_log_base. (Chenbo)

Maxime Coquelin (3):
  vhost: fix error path when setting memory tables
  vhost: fix fd leak in dirty logging setup
  vhost: fix fd leak in kick setup

 lib/librte_vhost/vhost_user.c | 74 +++++++++++++++++++++++------------
 1 file changed, 49 insertions(+), 25 deletions(-)

Comments

Maxime Coquelin Nov. 13, 2020, 8:39 a.m. UTC | #1 
On 11/12/20 6:10 PM, Maxime Coquelin wrote:
> This series fixes several leaks in Vhost-user requests
> handling.
> 
> Thanks to Xuan Ding from Intel for reporting these
> issues.
> 
> Changes in v4:
> - Fix wrong return value in set_mem_table (Xueming)
> 
> Changes in v3:
> - Remove the right closing of fds. (Chenbo)
> Changes in v2:
> - Fix typos in commit messages. (Chenbo)
> - Remove useless closing of fds in set_log_base. (Chenbo)
> 
> Maxime Coquelin (3):
>   vhost: fix error path when setting memory tables
>   vhost: fix fd leak in dirty logging setup
>   vhost: fix fd leak in kick setup
> 
>  lib/librte_vhost/vhost_user.c | 74 +++++++++++++++++++++++------------
>  1 file changed, 49 insertions(+), 25 deletions(-)
> 

Applied to dpdk-next-virtio/main.

Thanks,
Maxime