[dpdk-dev] [PATCH] net/virtio: Fix crash in virtio_dev_free_mbufs

Olivier Matz olivier.matz at 6wind.com
Mon Feb 5 09:17:22 CET 2018


On Sat, Feb 03, 2018 at 09:55:23AM -0500, David C Harton wrote:
> virtio_dev_free_mbufs was recently modified to free the
> virtqueues but failed to check whether the array was
> allocated.  Added a check to ensure vqs was non-null.
> 
> Fixes: bdb32afbb610 ("net/virtio: rationalize queue flushing")
> Cc: olivier.matz at 6wind.com
> 
> Signed-off-by: David C Harton <dharton at cisco.com>
> ---

Reviewed-by: Olivier Matz <olivier.matz at 6wind.com>

Thank you for this fix.
Out of curiosity, did you see the problem by code review or is there an
easy way to reproduce the crash?



More information about the dev mailing list