commit 9cc765559c39a299bdb55f3f7279abbcbe00a556
author Damjan Marion <damarion@cisco.com> Fri Mar 05 11:39:02 2021 +0100
committer Matthew Smith <mgsmith@netgate.com> Fri Mar 05 14:27:58 2021 +0000
tree 11db8da401668ffa41ca71824c54da0ecc656575
parent d89de31489cc0fb0c3d606fe9b30a97934942e80

avf: don't memcpy if adminq output buffer size is 0

Type: fix
Change-Id: I0df14ff87d0bf51eeb392f72434febf6c4a2957a
Signed-off-by: Damjan Marion <damarion@cisco.com>

src/plugins/avf/device.c

diff --git a/src/plugins/avf/device.c b/src/plugins/avf/device.c
index 139f1c9..fd17398 100644
--- a/src/plugins/avf/device.c
+++ b/src/plugins/avf/device.c
@@ -485,7 +485,7 @@
       goto done;
     }
 
-  if (d->flags & AVF_AQ_F_BUF)
+  if (out_len && d->flags & AVF_AQ_F_BUF)
     {
       void *buf = ad->arq_bufs + ad->arq_next_slot * AVF_MBOX_BUF_SZ;
       clib_memcpy_fast (out, buf, out_len);