Hello! > Function intel_sanity_check() calls conf1_read() which access d->domain > field. But intel_sanity_check() does not initialize this field and so > conf1_read() access some random data on stack. > > Tests showed that intel_sanity_check() always fails as in d->domain is > stored some non-zero number. > > Fix this issue by properly initializing struct pci_dev d and explicitly set > d->domain to zero in intel_sanity_check() as sanity check is verifying PCI > devices at domain 0. Thanks for finding the bug, but it makes little sense to initialize the fields twice (once by memset, second time by zeroing out individual members). Just keep the memset. Martin
