On 7/9/19, David Miller <davem@xxxxxxxxxxxxx> wrote: > From: Denis Kirjanov <kda@xxxxxxxxxxxxxxxxx> > Date: Tue, 9 Jul 2019 13:42:51 +0200 > >> @@ -962,7 +962,8 @@ static void vhost_del_umem_range(struct vhost_umem >> *umem, >> >> while ((node = vhost_umem_interval_tree_iter_first(&umem->umem_tree, >> start, end))) >> - vhost_umem_free(umem, node); >> + if (node) >> + vhost_umem_free(umem, node); > > If 'node' is NULL we will not be in the body of the loop as per > the while() condition. The patch is incorrect, please ignore > > How did you test this? >
