summaryrefslogtreecommitdiff
path: root/nuttx/drivers/usbdev
diff options
context:
space:
mode:
authorpatacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3>2012-04-25 16:47:28 +0000
committerpatacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3>2012-04-25 16:47:28 +0000
commitffcaa123f477ac9062398c06b68950203f0c899a (patch)
tree5ae0d3a4edccd428088dcf117a5241387c6d6f21 /nuttx/drivers/usbdev
parent798d0d0debae5b8dc0f40844adf18ce14906cfd7 (diff)
downloadpx4-nuttx-ffcaa123f477ac9062398c06b68950203f0c899a.tar.gz
px4-nuttx-ffcaa123f477ac9062398c06b68950203f0c899a.tar.bz2
px4-nuttx-ffcaa123f477ac9062398c06b68950203f0c899a.zip
Fix infinite loop in CDC/ACM driver
git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@4655 42af7a65-404d-4744-a932-0658087f49c3
Diffstat (limited to 'nuttx/drivers/usbdev')
-rw-r--r--nuttx/drivers/usbdev/cdcacm.c18
1 files changed, 10 insertions, 8 deletions
diff --git a/nuttx/drivers/usbdev/cdcacm.c b/nuttx/drivers/usbdev/cdcacm.c
index 2065b5433..24903b504 100644
--- a/nuttx/drivers/usbdev/cdcacm.c
+++ b/nuttx/drivers/usbdev/cdcacm.c
@@ -2182,14 +2182,16 @@ void cdcacm_uninitialize(FAR void *handle)
usbtrace(TRACE_CLSERROR(USBSER_TRACEERR_UARTUNREGISTER), (uint16_t)-ret);
}
- /* Unbind the class (if still bound) */
-
- if (priv->usbdev)
- {
- cdcacm_unbind(&drvr->drvr, priv->usbdev);
- }
-
- /* Unregister the driver (unless we are a part of a composite device) */
+ /* Unregister the driver (unless we are a part of a composite device). The
+ * device unregister logic will (1) return all of the requests to us then
+ * (2) all the unbind method.
+ *
+ * The same thing will happen in the composite case except that: (1) the
+ * composite driver will call usbdev_unregister() which will (2) return the
+ * requests for all members of the composite, and (3) call the unbind
+ * method in the composite device which will (4) call the unbind method
+ * for this device.
+ */
#ifndef CONFIG_CDCACM_COMPOSITE
usbdev_unregister(&drvr->drvr);