Messages in this thread | | | From | "Wang, Wei W" <> | Subject | RE: [PATCH v1] KVM: destruct kvm_io_device while unregistering it from kvm_io_bus | Date | Wed, 25 Jan 2023 02:38:12 +0000 |
| |
On Tuesday, January 24, 2023 7:25 AM, Sean Christopherson wrote: > > - kfree(bus); > > - return new_bus ? 0 : -ENOMEM; > > + kvm_iodevice_destructor(dev); > > + return 0; > > Unless I'm misreading things, this path leaks "bus".
Right, should keep the kfree above.
> Given that that intent is to send the fix for stable, that this is as much a > cleanup as it is a bug fix, and that it's not super trivial, I'm inclined to queue > my patch and then land this on top as cleanup.
Sounds good to me.
| |