On 26 Feb 2026, at 11:40, Eelco Chaudron via dev wrote:
> Coverity reports a data race where netdev_dpdk_vhost_destruct()
> accesses vhost_driver_flags without holding dpdk_mutex, while
> netdev_dpdk_vhost_client_reconfigure() writes to vhost_driver_flags
> with the mutex held (as is done 4 out of 4 times when writing).
>
> This could cause a race if another thread modifies vhost_driver_flags
> through netdev_dpdk_vhost_client_reconfigure() at the same time the
> destructor is running, potentially leading to incorrect socket cleanup.
>
> Fix by capturing the flag value while holding dpdk_mutex, similar to how
> vhost_id is already handled.
>
> Fixes: c1ff66ac80b5 ("netdev-dpdk: vHost client mode and reconnect")
> Signed-off-by: Eelco Chaudron <[email protected]>
Recheck-request: github-robot
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
- [ovs-dev] [PATCH 06/13] backtrace: Check fcntl() re... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 06/13] backtrace: Check f... Mike Pattrick via dev
- [ovs-dev] [PATCH 07/13] ovsdb-idl: Check ovsdb_datu... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 07/13] ovsdb-idl: Check o... Mike Pattrick via dev
- [ovs-dev] [PATCH 09/13] coverage: Fix data race whe... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 09/13] coverage: Fix data... Mike Pattrick via dev
- [ovs-dev] [PATCH 10/13] dpif-netdev: Fix race on st... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 10/13] dpif-netdev: Fix r... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 10/13] dpif-netdev: Fix r... Ilya Maximets
- [ovs-dev] [PATCH 11/13] netdev-dpdk: Fix vhost_driv... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 11/13] netdev-dpdk: Fix v... Eelco Chaudron via dev
- [ovs-dev] [PATCH 12/13] rconn: Fix max_backoff data... Eelco Chaudron via dev
- [ovs-dev] [PATCH 13/13] utilities: Add new words to... Eelco Chaudron via dev
- [ovs-dev] [PATCH 08/13] fatal-signal: Add mutex pro... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 08/13] fatal-signal: Add ... Mike Pattrick via dev
