Received: by 2002:a05:6a10:c604:0:0:0:0 with SMTP id y4csp3780331pxt; Tue, 10 Aug 2021 11:08:52 -0700 (PDT) X-Google-Smtp-Source: ABdhPJztrzuzf6AoX9Ol34Q3PoeJN7rU/ZPJYmfST655DfNTyx6VcH3d2xB7NbUQFgSGx6EKy1VP X-Received: by 2002:a05:6402:3552:: with SMTP id f18mr6306123edd.82.1628618932110; Tue, 10 Aug 2021 11:08:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1628618932; cv=none; d=google.com; s=arc-20160816; b=UG62oEu9wUBo1BxFKzw2TDL9IRHdmN2FfsvBjSQQH9KQtyOygj+g5azb39OyZC2lQ7 LPNpGSlJlIZ3UZJgkyWCVnDWkxUXBdTlx4hlQ0lmhwcgGznu8Ivc8C6jhWmycjEQypC4 VlbQoJ/1WcdWSI6M7Zcj4rP6D4n9XvtfIafSCEETHvKtRa5CPi4+NDepdtlKBrrKaeci 4zzjdOMQLa363c3w7dXHyNmb00tCi4DA3XP+uxId3wljBjMn2vbbJzWiHPcMfbuvSZ4J LWuusMq4dJdZt7H7pBN7B1oNYSGUDIA8EAGzbkKWaeBZPmOBBU1Vz8Y3zNNnXqR/IGlN ZR7g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=vHEHTNuZQZS6APru4MY7GBAGHOtrmaqhdlqOQggx71Y=; b=dFOwb5p9LNviL+e8RfFCYK+nAa1BMVJ18Y0MRr+ElDcBSgF+kbtjVBsSXOLYjKdDWN cDKN/pUtcot0xXgB4MjGmBGQmv3heRjAPBbby/1U0iR/Xt9zWx/SVD3zuUOyH9Q9WJ3V EO5T107b4Og94wHE2xf8s4LaqErUuvrXIrOgX/q0AzobqkMVTvW41TUKdLV24XHnmoof 1jyIspv9MuiqEkxW8SScVZKcZ+J6u1hySuilxEf9TYA9kB1nE36rmeHvUApWAuIu6NUH JstHV9OpUX7SnNZLOif6O/UxUET5hZ1ucfF4UM/PfNymAf2jgWkIi6+fndU6kYziU1Vr 8mIg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=JKJXq71r; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id rh8si6686537ejb.646.2021.08.10.11.08.27; Tue, 10 Aug 2021 11:08:52 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=JKJXq71r; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238467AbhHJSES (ORCPT + 99 others); Tue, 10 Aug 2021 14:04:18 -0400 Received: from mail.kernel.org ([198.145.29.99]:58908 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238608AbhHJR7t (ORCPT ); Tue, 10 Aug 2021 13:59:49 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 7547E61179; Tue, 10 Aug 2021 17:46:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1628617591; bh=MGfQA6lcM9g8o8bScDfSMHw06mk8qD9asDGzQiw1DAU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=JKJXq71rrERUEzlHscKDRLKoGOgah79e4j/8ca/TzGUeJ02UQkr6Fau2l+2/rKcly rzmQE4n9paep98JSDWi0nYt70AgFmm0FCpWQ6WaUNg9YACiowtKxQsmhfRmerlG7EY 3t1lIZWA80O9wNx0/kyYMximwadYLN70bpiHQbJg= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Peter Chen , Dmitry Osipenko Subject: [PATCH 5.13 099/175] usb: otg-fsm: Fix hrtimer list corruption Date: Tue, 10 Aug 2021 19:30:07 +0200 Message-Id: <20210810173004.217360223@linuxfoundation.org> X-Mailer: git-send-email 2.32.0 In-Reply-To: <20210810173000.928681411@linuxfoundation.org> References: <20210810173000.928681411@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Dmitry Osipenko commit bf88fef0b6f1488abeca594d377991171c00e52a upstream. The HNP work can be re-scheduled while it's still in-fly. This results in re-initialization of the busy work, resetting the hrtimer's list node of the work and crashing kernel with null dereference within kernel/timer once work's timer is expired. It's very easy to trigger this problem by re-plugging USB cable quickly. Initialize HNP work only once to fix this trouble. Unable to handle kernel NULL pointer dereference at virtual address 00000126) ... PC is at __run_timers.part.0+0x150/0x228 LR is at __next_timer_interrupt+0x51/0x9c ... (__run_timers.part.0) from [] (run_timer_softirq+0x2f/0x50) (run_timer_softirq) from [] (__do_softirq+0xd5/0x2f0) (__do_softirq) from [] (irq_exit+0xab/0xb8) (irq_exit) from [] (handle_domain_irq+0x45/0x60) (handle_domain_irq) from [] (gic_handle_irq+0x6b/0x7c) (gic_handle_irq) from [] (__irq_svc+0x65/0xac) Cc: stable@vger.kernel.org Acked-by: Peter Chen Signed-off-by: Dmitry Osipenko Link: https://lore.kernel.org/r/20210717182134.30262-6-digetx@gmail.com Signed-off-by: Greg Kroah-Hartman Signed-off-by: Greg Kroah-Hartman --- drivers/usb/common/usb-otg-fsm.c | 6 +++++- include/linux/usb/otg-fsm.h | 1 + 2 files changed, 6 insertions(+), 1 deletion(-) --- a/drivers/usb/common/usb-otg-fsm.c +++ b/drivers/usb/common/usb-otg-fsm.c @@ -193,7 +193,11 @@ static void otg_start_hnp_polling(struct if (!fsm->host_req_flag) return; - INIT_DELAYED_WORK(&fsm->hnp_polling_work, otg_hnp_polling_work); + if (!fsm->hnp_work_inited) { + INIT_DELAYED_WORK(&fsm->hnp_polling_work, otg_hnp_polling_work); + fsm->hnp_work_inited = true; + } + schedule_delayed_work(&fsm->hnp_polling_work, msecs_to_jiffies(T_HOST_REQ_POLL)); } --- a/include/linux/usb/otg-fsm.h +++ b/include/linux/usb/otg-fsm.h @@ -196,6 +196,7 @@ struct otg_fsm { struct mutex lock; u8 *host_req_flag; struct delayed_work hnp_polling_work; + bool hnp_work_inited; bool state_changed; };