On 10/19/23 09:24, David Marchand wrote:
On Mon, Sep 25, 2023 at 6:36 PM Maxime Coquelin
<[email protected]> wrote:

Acquiring the access lock is not enough to ensure
virtqueue's metadata such as vring pointers are valid.

The access status must also be checked.

Even if adding the lock was not enough, I would flag Fixes:
6c299bb7322f ("vhost: introduce vring call API")

Makes sense, adding it to v2.

Thanks,
Maxime


Fixes: c5736998305d ("vhost: fix missing virtqueue lock protection")
Fixes: 830f7e790732 ("vhost: add non-blocking API for posting interrupt")
Cc: [email protected]

Reported-by: Li Feng <[email protected]>
Signed-off-by: Maxime Coquelin <[email protected]>



Reply via email to