Commit a4d7fc8e authored Oct 31, 2017 by Chandana Kishori Chiluveru

USB: composite: Release spinlock before WARN()



if WARN() is called under spinlock, it will lead crash due to
watchdog timeout. Hence fix it by releasing spinlock before calling
WARN(). And also check cdev->config is NULL or not before calling
WARN().

Change-Id: Iebd00932b94164e9f40aa6126a2352437b1141ed
Signed-off-by: Vijayavardhan Vennapusa <vvreddy@codeaurora.org>
Signed-off-by: Chandana Kishori Chiluveru <cchiluve@codeaurora.org>

parent 7fffcd4e

drivers/usb/gadget/composite.c

+6 −0

Original line number	Diff line number	Diff line
		@@ -2545,7 +2545,13 @@ void usb_composite_setup_continue(struct usb_composite_dev *cdev)
		spin_lock_irqsave(&cdev->lock, flags);

		if (cdev->delayed_status == 0) {
		if (!cdev->config) {
		spin_unlock_irqrestore(&cdev->lock, flags);
		return;
		}
		spin_unlock_irqrestore(&cdev->lock, flags);
		WARN(cdev, "%s: Unexpected call\n", __func__);
		return;

		} else if (--cdev->delayed_status == 0) {
		DBG(cdev, "%s: Completing delayed status\n", __func__);