Re: [PATCH] virnodedeviceobj: Don't unlock virNodeDeviceObj in virNodeDeviceObjListRemove()

2022-02-01 Thread Michal Prívozník
On 2/1/22 15:43, Michal Privoznik wrote: > When virNodeDeviceObjListRemove() is called, the passed > virNodeDeviceObj is removed from internal list of node devices > and then unrefed and unlocked. While the former is warranted (the > object was refed at the beginning of the function) the unlock is

[PATCH] virnodedeviceobj: Don't unlock virNodeDeviceObj in virNodeDeviceObjListRemove()

2022-02-01 Thread Michal Privoznik
When virNodeDeviceObjListRemove() is called, the passed virNodeDeviceObj is removed from internal list of node devices and then unrefed and unlocked. While the former is warranted (the object was refed at the beginning of the function) the unlock is not. In fact, it's wrong from conceptual POV. We