mbox series

[0/7] vhost: make VQ metadata dereferencing robust

Message ID 20201019173415.582407-1-maxime.coquelin@redhat.com (mailing list archive)
Headers
Series vhost: make VQ metadata dereferencing robust |

Message

Maxime Coquelin Oct. 19, 2020, 5:34 p.m. UTC 
  The series aims at improving virtqueue metadata pointers
dereferencing robust.

First patch fixes a real issue reported by our QE.

Five next patches validates queue index passed as input
of the Vhost API. They are split in 5 patches to ease
backporting to LTSes.

Last patch is not mandatory now that patch 1 ensures there
won't be holes in the array of virtqueue metadata pointers,
but I think it is a nice to have anyway.

Maxime Coquelin (7):
  vhost: fix virtqueues metadata allocation
  vhost: validate index in available entries API
  vhost: validate index in guest notification API
  vhost: validate index in live-migration API
  vhost: validate index in inflight API
  vhost: validate index in async API
  vhost: check virtqueue metadata pointer

 lib/librte_vhost/vhost.c      | 76 ++++++++++++++++++++++++++++-------
 lib/librte_vhost/vhost_user.c | 12 ++++++
 2 files changed, 74 insertions(+), 14 deletions(-)

Comments

Maxime Coquelin Oct. 23, 2020, 11:21 a.m. UTC | #1 
On 10/19/20 7:34 PM, Maxime Coquelin wrote:
> The series aims at improving virtqueue metadata pointers
> dereferencing robust.
> 
> First patch fixes a real issue reported by our QE.
> 
> Five next patches validates queue index passed as input
> of the Vhost API. They are split in 5 patches to ease
> backporting to LTSes.
> 
> Last patch is not mandatory now that patch 1 ensures there
> won't be holes in the array of virtqueue metadata pointers,
> but I think it is a nice to have anyway.
> 
> Maxime Coquelin (7):
>   vhost: fix virtqueues metadata allocation
>   vhost: validate index in available entries API
>   vhost: validate index in guest notification API
>   vhost: validate index in live-migration API
>   vhost: validate index in inflight API
>   vhost: validate index in async API
>   vhost: check virtqueue metadata pointer
> 
>  lib/librte_vhost/vhost.c      | 76 ++++++++++++++++++++++++++++-------
>  lib/librte_vhost/vhost_user.c | 12 ++++++
>  2 files changed, 74 insertions(+), 14 deletions(-)
> 

Applied to dpdk-next-virtio/main.

Thanks,
Maxime