Received: by 2002:a05:7412:8521:b0:e2:908c:2ebd with SMTP id t33csp1095338rdf; Sat, 4 Nov 2023 06:26:12 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHvc3ibbC85wv+5hGQkTF8Jw9aM3On1Fwbx19QmiyaLkVBrlsoUi+TfUgcqC9GghJh7eJFb X-Received: by 2002:a05:6a00:1394:b0:6bc:3885:4a9c with SMTP id t20-20020a056a00139400b006bc38854a9cmr25604820pfg.23.1699104372226; Sat, 04 Nov 2023 06:26:12 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1699104372; cv=none; d=google.com; s=arc-20160816; b=lX8TuXLVfX7FhpLRTIoZeWyRSqezUp/EVBBEINTLVgUFbJxtOXt2nVAnXlPint2PXu rM935fUJuwkIGCIC1SBv0igF3LgOMlx9niM+OTx1/X7dQYU3GS6kZC4VEYz8BOMjouWC LxYQPsoSSrdGDL+kmLlqSaQNOsrAPblSFgd9s91rHBLadmKpt3sSA+IjfPjlLEECad0K 6vTtYV+OPofwhq9G2clFrFyycAdBhYxtWeUyiCNKiNRV8c9yRTR+m7Nd3/YNDN3zKRdD EwAD/f5K5pgS/9At9OH0MtL7cEJywAlWtolDfmqp484JravabeCbCwQYrH1FbGP0NYvT IM1g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:message-id:references :in-reply-to:subject:cc:to:from:date:mime-version; bh=rXQGs7KQugKSpemLofTIqQwhp7a5bLcV9u3Y/f+1TRc=; fh=NnvjNDOWs0Ks7KaawB8y11lPKsizyYHpVg0uvmrHSnc=; b=NI0LUi/X+IReQYj/FoAXXna65PpiIu+UALnwjPiFAnYIz9q/x8FkLPsxs3stUoKUPY vZwzXbYvI+5cFwtqUOMAq/h4BdgaW/KclpZzxUu1BBMOfjOLHgMXL8PeZzwF0FIpCIty 8wAWBeBclF9zRVf5aHhvaU5EY14qib4QA0ctJ97hkunuMOqoQUB55lfC36YENojJNSez HR5gNik7FCcweVUTpkYIsCprRF9AuMgNq7P0ren7FfE0qvb+f7nvXN0lOgWfiFnoODDy aGUBKUveCtfMUV17axgDfJ+lokoQtzdGNkPbmH0R2WQ4yyguaqf/E66kRUtJDArlM6/e Vh9g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from fry.vger.email (fry.vger.email. [2620:137:e000::3:8]) by mx.google.com with ESMTPS id j4-20020a056a00130400b006c0fe409cafsi3744831pfu.153.2023.11.04.06.26.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 04 Nov 2023 06:26:12 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by fry.vger.email (Postfix) with ESMTP id 47E78809F3B0; Sat, 4 Nov 2023 06:26:09 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at fry.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232186AbjKDN0C (ORCPT + 99 others); Sat, 4 Nov 2023 09:26:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47054 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230151AbjKDN0B (ORCPT ); Sat, 4 Nov 2023 09:26:01 -0400 Received: from connect.vanmierlo.com (fieber.vanmierlo.com [84.243.197.177]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8FBACE3; Sat, 4 Nov 2023 06:25:55 -0700 (PDT) X-Footer: dmFubWllcmxvLmNvbQ== Received: from roundcube.vanmierlo.com ([192.168.37.37]) (authenticated user m.brock@vanmierlo.com) by connect.vanmierlo.com (Kerio Connect 10.0.2 patch 1) with ESMTPA; Sat, 4 Nov 2023 14:25:51 +0100 MIME-Version: 1.0 Date: Sat, 04 Nov 2023 14:25:51 +0100 From: m.brock@vanmierlo.com To: Florian Eckert Cc: Eckert.Florian@googlemail.com, gregkh@linuxfoundation.org, jirislaby@kernel.org, pavel@ucw.cz, lee@kernel.org, kabel@kernel.org, u.kleine-koenig@pengutronix.de, linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org, linux-leds@vger.kernel.org Subject: Re: [Patch v6 3/7] leds: ledtrig-tty: free allocated ttyname buffer on deactivate In-Reply-To: <20231030100447.63477-4-fe@dev.tdt.de> References: <20231030100447.63477-1-fe@dev.tdt.de> <20231030100447.63477-4-fe@dev.tdt.de> Message-ID: <6be2d8101fb8da3455974ead01721e7e@vanmierlo.com> X-Sender: m.brock@vanmierlo.com Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-0.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on fry.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (fry.vger.email [0.0.0.0]); Sat, 04 Nov 2023 06:26:09 -0700 (PDT) Florian Eckert wrote on 2023-10-30 11:04: > The ttyname buffer for the ledtrig_tty_data struct is allocated in the > sysfs ttyname_store() function. This buffer must be released on trigger > deactivation. This was missing and is thus a memory leak. Shouldn´t there be a Fixes tag here? And as a side note: you have patches 1..7/7 and a cover letter 0/8 ? Maarten