Hi On Fri, Jan 29, 2016 at 4:43 PM, Markus Armbruster <arm...@redhat.com> wrote: > The existing users pass a func that saves dev, and free the saved dev > later. Works as long as we call func() at most once. If multiple > devices match, all but the last one are leaked. Can this happen?
It is the responsability of the func() callback to deal with multiple matches. I don't think this needs to change. This fix is only about the case of unmatching devices that need to be free within qpci_device_foreach(). Do you ack that fix? -- Marc-André Lureau