Coverity warns about the following code in vdev_queue_io_to_issue:
zio = avl_nearest(tree, idx, AVL_AFTER);
if (zio == NULL)
zio = avl_first(tree);
ASSERT3U(zio->io_priority, ==, p);
Coverity "thinks" that zio can still be NULL after avl_first().
I think that that can be true if vdev_queue_io_to_issue() is called from
vdev_queue_io_done() and the corresponding tree is empty, but I am not sure if
that's possible.
Thank you.
--
Andriy Gapon
-------------------------------------------
openzfs-developer
Archives: https://www.listbox.com/member/archive/274414/=now
RSS Feed: https://www.listbox.com/member/archive/rss/274414/28015062-cce53afa
Modify Your Subscription:
https://www.listbox.com/member/?member_id=28015062&id_secret=28015062-f966d51c
Powered by Listbox: http://www.listbox.com
