15 Dec
2023
15 Dec
'23
12:25 p.m.
On Fri, Dec 15, 2023 at 11:20:10AM +0000, Daniel P. Berrangé wrote:
Libvirt is executing the error callback and work done in the error callback is breaking our API guarantee.
This broken API guarantee is not the app's fault, it is libvirt's fault for not protecting itself against things the callback might do.
IOW, should we fix the virDispatchError function, to do the save/restore dance either side of invoking the callback. This would aviod exposing any new APIs, and make the app should "just work".
This would certainly fix my scenario, yes, thanks. We still have to deal with the fact that e.g. virDomainFree() cannot be used in cleanup paths, but the workarounds there are generally not so bad. regards john