From d29cb09da37b76a04095103ddfe59219c8e5c6b3 Mon Sep 17 00:00:00 2001 From: Gregory Nutt Date: Wed, 18 Oct 2017 11:06:14 -0600 Subject: [PATCH] drivers/usbdev: Move test for NULL pointer before the pointer is deferences. Noted by Juha Niskanen. --- drivers/usbdev/cdcacm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/usbdev/cdcacm.c b/drivers/usbdev/cdcacm.c index 146c3c17ce5..7593156a916 100644 --- a/drivers/usbdev/cdcacm.c +++ b/drivers/usbdev/cdcacm.c @@ -480,10 +480,11 @@ static int cdcacm_recvpacket(FAR struct cdcacm_dev_s *priv, uint16_t nexthead; uint16_t nbytes = 0; + DEBUGASSERT(priv != NULL && rdcontainer != NULL); + uinfo("head=%d tail=%d nrdq=%d reqlen=%d\n", priv->serdev.recv.head, priv->serdev.recv.tail, priv->nrdq, reqlen); - DEBUGASSERT(priv != NULL && rdcontainer != NULL); #ifdef CONFIG_CDCACM_IFLOWCONTROL DEBUGASSERT(priv->rxenabled && !priv->iactive); #else