From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 85D422F067E for ; Tue, 18 Nov 2025 06:46:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763448399; cv=none; b=copU1CjENTB+u+ME2NlzBee2HNRXHaIMoyWkel0HU4Pqlr5BUUGcdmpo7Md/gqmtTKn2s8z1/LmmOAPyCNVYdwmdI7+Vk/P/jixTEEstRJtZcihnOiDbHgMgWV4eAfwDKPV6vKkfh1gP603IPT6az2DrFUh7hCyAc4o5DePrNwk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763448399; c=relaxed/simple; bh=acujsx5P9AhRcG+rLz3vEfDBti6lUvFh4LWYV3tGlm8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Vm9VpJGmET6MH20D2VJft+h6EYEn+aZk1JdPsvZzMF78tr1QWGmyRQjT0VayJQThYSGjBb1X8Yi1PYPMPlLo/c7rpclJpNfN3xeAvD5a/35pILEZE+euZWi5UDgE46MPCTonbsO4b0FZ7p2+cdHfbjMHp55VDb2CQsECA3RTGn4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=o4EXfk9N; arc=none smtp.client-ip=209.85.128.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="o4EXfk9N" Received: by mail-wm1-f47.google.com with SMTP id 5b1f17b1804b1-47790b080e4so19646985e9.3 for ; Mon, 17 Nov 2025 22:46:37 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1763448396; x=1764053196; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=bV81dIpmp2beeLjKl6yrCue5z+eyzcey/0i+ySpQ3bY=; b=o4EXfk9N0TzPjImeauhSC7o07RtfBQLBgPaXzRU9jzye9/Z5UF8NDgW+bpHkfgvRzG OX6podmX7/zDTEDuupHrMynWdvOpiODQUvEZYQ4jtVHqW7cJ2nXUOqC0Jn/0gffcdJk5 D00q8jULEUetcIwA+m/EJ60C8YEbkyy5fd3n/Mg1+Bxo7YL5GqgVFEEDu/HzTRJLZRJg iQCKztVaavWTvT03DMjphosMXhE/CPsN8WbmHKHdO1lF0VDEdgw7ASH1uvsqd8gakHRo JVytF+OWdkUn5opfRcAFKs7FdKvX5G5OXi5uELSWBj1QhpVCMZxU1sqSOFkQdiwWtxYN 2kIg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763448396; x=1764053196; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=bV81dIpmp2beeLjKl6yrCue5z+eyzcey/0i+ySpQ3bY=; b=aJ7ye3BD8UgFumsiqcXaX2quKq+k75UZ0JENwieN0m+B9UNZFO/ECGkz4f7J9exKpG ofjR2Il1twFfGYnXVeKkKFLEHnscmpvYer4vZJtqEWfH9NxYF7JRFr8uDIKqO6RKOwYD v8JZDDW8hocYqR/UPDCb9iqCGBZKPTzKo/rHgygO4O8BY1FrBUV2mBCLw5V1tHicZLYj 2FAIdOgCfLK+e9DvswhzqvCPzhw+e91GPScGaUji8RE/GA2rqx7io05RBl9PAC0jyBKX bxX3j2Rn/jNph5sY+1I5O76RTf2rwt+v/+M70Eb4cdaXGoTodxMDWdANbMKnNnEq//vW vjsg== X-Forwarded-Encrypted: i=1; AJvYcCV3cDcLFi0wlIYBUkCz7n8N92uwVxcErFpIO0I1ClTdCT+bsf2lJIV9CjSUM68Pkwc+EMPcm7sT9ytd/Ys=@vger.kernel.org X-Gm-Message-State: AOJu0YxTM62R+3pW5lIdjtYtWTzHwhxp5ks01LJpQkUT6/i/xfZZqLj4 QlJd3SfqW3A7vjK4nT2/qma+MTs3td3AfFiZIzzBk+6wLQVw6MJSm22FfOO3a45fjsQ= X-Gm-Gg: ASbGnctrlMNk60mMKSsbeoA4Hm40c89ksoNKW+bWye+Pf146Pu8T0HDTVY4c91r/3BB 1I296Qty7n7tTKNzW3FE1ugOIVsU/Y0EXl1iVnhCE1C9ZJewkJDsYVmFXUUqAZyLa5SN+XOq47P VYjK2RFKzwwum/bPi/csf1Myx5W+2a5aZF+BxFknYH143+qx1hy21J+/4GsbCABmoqfSZMvlPAc 3BtImE7KtUAe0h6gBNXil6W4c/tUlP/4X+/o5dJ93/smqO2kAH4JjpR72fIUpSk5n5IHv60TtO6 5yZq38KgVl20UxyVQiyZO4lMUtUTQoL5h4sQlA3gFOI3G14LBnio0pEJjL0r8YdZpVtGHeBde9Q WZMlRBBl8lcw3YA83z+yAErJiwAepUG+U7LF/W2IZqCieWpix1HqAvxZy/OU/ZXgZUMg/LkLuO7 UTs7eagA== X-Google-Smtp-Source: AGHT+IHq6Tms2Tw4FCmoIgazMAaNdHCOZjsBWCUnSI/7xQlYbP/8sN+ivg/uC6ukSMsCFy3UUZ450Q== X-Received: by 2002:a05:600c:4744:b0:45c:4470:271c with SMTP id 5b1f17b1804b1-4778fe6be5fmr138575145e9.18.1763448395731; Mon, 17 Nov 2025 22:46:35 -0800 (PST) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with UTF8SMTPSA id 5b1f17b1804b1-477a9e19875sm8557925e9.16.2025.11.17.22.46.34 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 17 Nov 2025 22:46:35 -0800 (PST) Date: Tue, 18 Nov 2025 09:46:30 +0300 From: Dan Carpenter To: "Korenblit, Miriam Rachel" Cc: Ally Heev , Johannes Berg , "linux-wireless@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v2] net: wireless: fix uninitialized pointers with free attribute Message-ID: References: <20251107-aheev-uninitialized-free-attr-wireless-v2-1-674fc3e5c78e@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@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, Nov 10, 2025 at 01:17:10PM +0000, Korenblit, Miriam Rachel wrote: > > Subject: [PATCH v2] net: wireless: fix uninitialized pointers with free attribute > > Please send iwlwifi patches to either iwlwifi-next or iwlwifi-fixes ([PATCH iwlwifi-next] wifi: iwlwifi: blah blah ) > In your case it is not really fixing any bug, since we never return from the functions without initializing the pointers. I'm not a fan of these subsystem rules. If you're working over the entire kernel then you would have to track over 400 different trees. This could easily be done in the one place on the recieving end in the QC process instead of forcing every contributor to write little shell scripts. Anyway here is the code how I handle that for netdev. For BPF, which is the only other subsystem with this rule, I only send bug reports instead of patches because I really want to discourage this kind of rule... git remote update net git remote update net-next if grep -q netdev $MAIL_FILE && ! grep -q wireless $MAIL_FILE ; then if [ "$FIXES_COMMIT" != "" ] ; then if git merge-base --is-ancestor $FIXES_COMMIT net/main ; then TREE="net" elif git merge-base --is-ancestor $FIXES_COMMIT net-next/main ; then TREE="net-next" else TREE="net-other" fi else TREE="net-next" fi fi So the new iwlwifi code would be (free handed in my email client). First add iwlwifi as a remote to your developement tree. git remote add iwlwifi https://git.kernel.org/pub/scm/linux/kernel/git/iwlwifi/iwlwifi-next.git git remote update iwlwifi if grep -q -w /iwlwifi/ $MAIL_FILE ; then if [ "$FIXES_COMMIT" != "" ] ; then if git merge-base --is-ancestor $FIXES_COMMIT iwlwifi/fixes ; then TREE="iwlwifi" elif git merge-base --is-ancestor $FIXES_COMMIT iwlwifi/next ; then TREE="iwlwifi-next" else TREE="iwlwifi-other" fi else TREE="iwlwifi-next" fi fi regards, dan carpenter