On Wed, Sep 12, 2018 at 11:12:50AM +0200, Maxime Coquelin wrote:
On 09/06/2018 08:19 PM, Jens Freimann wrote:
[...]
+ if (vtpci_packed_queue(vq->hw)) { + i = vq->vq_used_cons_idx; + while (desc_is_used(&descs[i], &vq->vq_ring) && + i < vq->vq_nentries) {It may be preferable to ensure 'i' is within the ring size before using it in desc_is_used(). And raise (at least) an error message if it isn't.
Yes, I'll fix this. Thanks! regards,Jens