From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8D73FC433EF for ; Fri, 8 Apr 2022 20:58:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239562AbiDHVAi (ORCPT ); Fri, 8 Apr 2022 17:00:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37740 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229586AbiDHVAg (ORCPT ); Fri, 8 Apr 2022 17:00:36 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 92DF139142 for ; Fri, 8 Apr 2022 13:58:31 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 2779261F49 for ; Fri, 8 Apr 2022 20:58:31 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3BAC2C385A3; Fri, 8 Apr 2022 20:58:29 +0000 (UTC) Date: Fri, 8 Apr 2022 16:58:27 -0400 From: Steven Rostedt To: Thomas Gleixner Cc: Linus Torvalds , LKML , jstultz@google.com, Stephen Boyd , Andrew Morton , Peter Zijlstra , Marcel Holtmann , Johan Hedberg , Luiz Augusto von Dentz , Eric Dumazet , Guenter Roeck Subject: Re: [RFC][PATCH] timers: Add del_time_free() to be called before freeing timers Message-ID: <20220408165827.42475fb2@gandalf.local.home> In-Reply-To: <87v8vjiaih.ffs@tglx> References: <20220407161745.7d6754b3@gandalf.local.home> <87pmlrkgi3.ffs@tglx> <87v8vjiaih.ffs@tglx> X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 08 Apr 2022 22:29:58 +0200 Thomas Gleixner wrote: > What a shutdown function would prevent here is UAF, but I'm not entirely > sure whether it will simplify coordinated shutdown and remove the > requirement of a priv->shutdown flag all over the place. It might make > some of the driver muck just get stuck in the shutdown, but that's > definitely an improvement over a potential UAF which happens every blue > moons. Note, it is the cause of a large percentage of crash reports reported by ChromeOS. And we do not even know if it was this bluetooth issue that caused them. There's evidence they are mostly caused by the wifi code. I only used the bluetooth issue because it was the first one we found that looked obviously wrong. -- Steve