This follows up the inquiry at [1], where Will and Robin agreed that the two list iterator sites in drivers/iommu/ that run past the loop end are worth fixing.
Both loops only walk a list to find a list_head to insert before, not to operate on the entries. Per Robin's suggestion, rather than tracking an explicit insert_before pointer, each loop now iterates with list_for_each() and keeps the list_entry() dereference inside the loop body. There is no functional change in either patch. [1] https://lore.kernel.org/all/CAHPEe=G-FZvEXjkE+vAXN5MHXCtsOaUoKwg2RQL6m=om+c2...@mail.gmail.com/ Maoyi Xie (2): iommu: Avoid using the list iterator past the loop in iommu_insert_resv_region() iommu/virtio: Avoid using the list iterator past the loop in viommu_add_resv_mem() drivers/iommu/iommu.c | 7 +++++-- drivers/iommu/virtio-iommu.c | 10 +++++++--- 2 files changed, 12 insertions(+), 5 deletions(-) base-commit: 254f49634ee16a731174d2ae34bc50bd5f45e731 -- 2.34.1
