[PATCH] virnwfilterobj: Don't use virObjectLockGuard() with virNWFilterObj

Erik Skultety eskultet at redhat.com
Fri Mar 18 12:54:08 UTC 2022


On Fri, Mar 18, 2022 at 11:43:34AM +0100, Michal Privoznik wrote:
> While its name would suggest that virNWFilterObj is an actual
> virObject it is not. It's a plain structure (with virMutex as its
> first member). Therefore, when locking the struct
> virObjectLockGuard() can' be used and virLockGuardLock() must be
> used instead.
> 
> Spotted-by: Martin Kletzander <mkletzan at redhat.com>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---

This fixes the crasher in the integration CI we saw yesterday:
Reviewed-by: Erik Skultety <eskultet at redhat.com>

After this patch, there's still a deadlock [1], but you should push this fix in
the meantime.

[1] https://paste.centos.org/view/6a7bfd66



More information about the libvir-list mailing list