----- Original Message ----- From: "Eric Blake" <eblake@xxxxxxxxxx> False positive. Just a few lines earlier, in loop 4, we guaranteed that dev was added to driver->activePciHostdevs, therefore, activeDev cannot be NULL here. That said, we could probably simplify things by consolidating loop 5 and 6 into one, and in the process of that simplification, silence the > it should consolidate loop 4 and loop 5 into one :-)however, loop 6 steps > are a clear design if don't consider Coverity complains. Alex spurious warning from the static analyzer. -- Eric Blake eblake@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list