From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EFE671D6194; Tue, 3 Sep 2024 07:02:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725346971; cv=none; b=eSqPeGHn8eDvIW707N3MVVQCg8Rwy2E+nuLZyUNrYyGYex0cMsS/4ULkdu1wfN8bABrXUV6b4Ud/wrMdpUsV0jteCLGOVa130XKAT1z+e2loyH9KMsV6ZWujt0bIWheSIjWfL8Q3wL/vJCmgYEJ2vIozqumRT4VY9phUENSuXXY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1725346971; c=relaxed/simple; bh=3qidJfQt+X4a5kMK7cfyTY7OkqPL4BDhOdGy6okPpKo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=GPWpWMJGC0cnSTe09COhtgLzHi8QJBT2J8YGuXzwVJaxy2/Fo4wf9TKAVs9SymLndCfZ13O3UBRyQMTM1icusUelOOXzEOq3+FkTj3cTkk2fhyyJJubmAwahSts7O/QYgweeoNLn3yMPJdMWfbB7tZnsmVHX8bCfe+O+JEdcUeI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=UzM/z1mP; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="UzM/z1mP" Received: by smtp.kernel.org (Postfix) with ESMTPSA id F1B82C4CEC5; Tue, 3 Sep 2024 07:02:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1725346970; bh=3qidJfQt+X4a5kMK7cfyTY7OkqPL4BDhOdGy6okPpKo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=UzM/z1mPycuq4e2yNt8f7GA15sih3ir21E8Talxo4MlWCSmmpSmTueSUhufJ/kTq/ n+7dvsgFyKe1KYXdSL/FJVfg9Szkry1FA46kTuCvhTdTDOs5uipkLfz53z/DQgoQ08 LyVDpiIkc5hCdxJEAHOh0Nc1jEjY8ksZQOl3xIqc= Date: Tue, 3 Sep 2024 09:02:47 +0200 From: Greg Kroah-Hartman To: Heikki Krogerus Cc: stable@vger.kernel.org, linux-usb@vger.kernel.org, Charles Yo , Kyle Tso , Amit Sunil Dhamne , Ondrej Jirman , Dmitry Baryshkov Subject: Re: [PATCH 6.6] usb: typec: fix up incorrectly backported "usb: typec: tcpm: unregister existing source caps before re-registration" Message-ID: <2024090336-spider-skirmish-2ac9@gregkh> References: <2024083008-granddad-unmoving-828c@gregkh> Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Mon, Sep 02, 2024 at 09:25:44AM +0300, Heikki Krogerus wrote: > On Fri, Aug 30, 2024 at 04:00:09PM +0200, Greg Kroah-Hartman wrote: > > In commit b16abab1fb64 ("usb: typec: tcpm: unregister existing source > > caps before re-registration"), quilt, and git, applied the diff to the > > incorrect function, which would cause bad problems if exercised in a > > device with these capabilities. > > > > Fix this all up (including the follow-up fix in commit 04c05d50fa79 > > ("usb: typec: tcpm: fix use-after-free case in > > tcpm_register_source_caps") to be in the correct function. > > > > Fixes: 04c05d50fa79 ("usb: typec: tcpm: fix use-after-free case in tcpm_register_source_caps") > > Fixes: b16abab1fb64 ("usb: typec: tcpm: unregister existing source caps before re-registration") > > Reported-by: Charles Yo > > Cc: Kyle Tso > > Cc: Amit Sunil Dhamne > > Cc: Ondrej Jirman > > Cc: Heikki Krogerus > > Cc: Dmitry Baryshkov > > Signed-off-by: Greg Kroah-Hartman > > Acked-by: Heikki Krogerus Thanks for the review!