On Wed, Jul 01, 2015 at 11:00:45AM +0100, Andrew Cooper wrote: > On 01/07/15 00:20, [email protected] wrote: > > --- a/xen/drivers/passthrough/vtd/dmar.c > > +++ b/xen/drivers/passthrough/vtd/dmar.c > > @@ -81,6 +81,13 @@ static int __init acpi_register_rmrr_unit(struct > > acpi_rmrr_unit *rmrr) > > return 0; > > } > > > > +static void scope_devices_free(struct dmar_scope *scope) > > +{ > > + if ( scope ) > > + scope->devices_cnt = 0; > > + xfree(scope->devices); > > This is very liable to suffer a NULL pointer dereference.
Thanks Andrew, reposting. > > ~Andrew _______________________________________________ Xen-devel mailing list [email protected] http://lists.xen.org/xen-devel
