sergiotarxz/nuttx
sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

drivers/usbdev/cdcacm.c: Avoid using priv before checking its validity.

Browse Source
This commit is contained in:
Juha Niskanen 2017-09-26 06:31:30 -06:00 committed by Gregory Nutt
parent b5123603fd
commit d141242a25
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/usbdev/cdcacm.c
View File

@ -640,8 +640,6 @@ static int cdcacm_serialstate(FAR struct cdcacm_dev_s *priv)
irqstate_t flags; irqstate_t flags;
int ret; int ret;
usbtrace(CDCACM_CLASSAPI_FLOWCONTROL, (uint16_t)priv->serialstate);
DEBUGASSERT(priv != NULL && priv->epintin != NULL); DEBUGASSERT(priv != NULL && priv->epintin != NULL);
#ifdef CONFIG_DEBUG_FEATURES #ifdef CONFIG_DEBUG_FEATURES
if (priv == NULL || priv->epintin == NULL) if (priv == NULL || priv->epintin == NULL)
@ -651,6 +649,8 @@ static int cdcacm_serialstate(FAR struct cdcacm_dev_s *priv)
} }
#endif #endif
usbtrace(CDCACM_CLASSAPI_FLOWCONTROL, (uint16_t)priv->serialstate);
flags = enter_critical_section(); flags = enter_critical_section();
/* Use our interrupt IN endpoint for the transfer */ /* Use our interrupt IN endpoint for the transfer */