On 6/23/25 12:22, Zhenzhong Duan wrote:
When open /dev/vfio/vfio fails, SIGSEGV triggers because vfio_listener_unregister() doesn't support a NULL bcontainer pointer.Fixes: a1f267a7d4d9 ("vfio/container: reform vfio_container_connect cleanup") Signed-off-by: Zhenzhong Duan <[email protected]> --- hw/vfio/container.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/hw/vfio/container.c b/hw/vfio/container.c index 3e8d645ebb..2853f6f08b 100644 --- a/hw/vfio/container.c +++ b/hw/vfio/container.c @@ -710,7 +710,9 @@ static bool vfio_container_connect(VFIOGroup *group, AddressSpace *as, return true;fail:- vfio_listener_unregister(bcontainer); + if (new_container) { + vfio_listener_unregister(bcontainer); + }if (group_was_added) {vfio_container_group_del(container, group);
Applied to vfio-next. Thanks, C.
