Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp351525yba; Wed, 15 May 2019 02:29:03 -0700 (PDT) X-Google-Smtp-Source: APXvYqy4FD/Xq2pI5basrfjcoNPUmRsZzijGS+Cl432whrIpl86Jv+p6fSR4vmEgLyzL4GCWzCx/ X-Received: by 2002:a17:902:263:: with SMTP id 90mr43321920plc.257.1557912543669; Wed, 15 May 2019 02:29:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1557912543; cv=none; d=google.com; s=arc-20160816; b=hnaEdpPrscnMNvRfOZ2F9OA+cYEbeTTBEIscCGcY6nmR5zJ4F5PkkW8WDet5dnqeQ+ g9ApOV8yzVRF9nZuWwDPyOGGnl6geZ4pGQi+h54OaUAiAHUbQUacn4pbp2A2ZE5DDOx0 9FdV8D2d4A9v0UoNVRDBseHOmymbRtkyNOpIOb/hCH0lLlQY8hrd0Kto6DfsekKtY/K+ 1WwPkucJ5qOrfU2KPR1QX6JiTVcpSWJ+X/TFOqwbFM5ZfZ6f5FzkZQZAel5UPZIFuM0a 1gf+kH+zchgAVJDvgIzn9/RwgRAz5xWyjgX590PGXn87hOhNZf30ryBdQ3PSeUTprnkD neaA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature; bh=3qRNFOXkrkponxLfzGmEjoSRxN5kc1zj/UiLZP1QB8Q=; b=xIGGBeaX3+p2fG2AWiM8BBKsnSmTuj63n7j23th3vYgA//27YSDH5sJjGVKb7JT+nb xOktNpqJuAPT4Pduia+RcVO2HgZD6Q0WQPFfwipWejX+26d1ZpMZYV0GGlZUoI/hIG65 N531l8niPHdkXf9rLHffnJBZFOlDCh+x7bTN9jWejf2f4dsjIAAM3wj0DdL7ITQGdUNi eM/NCqS8GjrpFNqXBxjE1mRM1Rv6Q5Gf0j1ouJFClM3WPIRP5MSrj/2yP5BMsRqpKfQC jtvqozpJKWhnT3wc9sT3wCzmGpdG5ZPX+rkIAfNm1yr/qWNLcZPd/+Wvb+xZctRWWi/o dxxQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@c-s.fr header.s=mail header.b=c4JFbjzR; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 19si1315956pgu.457.2019.05.15.02.28.48; Wed, 15 May 2019 02:29:03 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@c-s.fr header.s=mail header.b=c4JFbjzR; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726290AbfEOJ0K (ORCPT + 99 others); Wed, 15 May 2019 05:26:10 -0400 Received: from pegase1.c-s.fr ([93.17.236.30]:47568 "EHLO pegase1.c-s.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725871AbfEOJ0J (ORCPT ); Wed, 15 May 2019 05:26:09 -0400 Received: from localhost (mailhub1-int [192.168.12.234]) by localhost (Postfix) with ESMTP id 453q1C1rtYz9vDc8; Wed, 15 May 2019 11:26:07 +0200 (CEST) Authentication-Results: localhost; dkim=pass reason="1024-bit key; insecure key" header.d=c-s.fr header.i=@c-s.fr header.b=c4JFbjzR; dkim-adsp=pass; dkim-atps=neutral X-Virus-Scanned: Debian amavisd-new at c-s.fr Received: from pegase1.c-s.fr ([192.168.12.234]) by localhost (pegase1.c-s.fr [192.168.12.234]) (amavisd-new, port 10024) with ESMTP id 1Uv3Awwze50y; Wed, 15 May 2019 11:26:07 +0200 (CEST) Received: from messagerie.si.c-s.fr (messagerie.si.c-s.fr [192.168.25.192]) by pegase1.c-s.fr (Postfix) with ESMTP id 453q1C0TlNz9vDc7; Wed, 15 May 2019 11:26:07 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=c-s.fr; s=mail; t=1557912367; bh=3qRNFOXkrkponxLfzGmEjoSRxN5kc1zj/UiLZP1QB8Q=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=c4JFbjzRxT7TGL7+nB5rqpyUQRiyXqf/27PlDer9jVEZnqqGKWqvy49aA7/bWJcNy 58RDcE9gRAImFqhpJvA2pjjE7c11hb3wx8OxM5aLCys2ozH1BIFJFazMgXgeQINwVU U8uooYdTdm7qVGbyGgLGLIlxo+e8C6QIlM4aSd+o= Received: from localhost (localhost [127.0.0.1]) by messagerie.si.c-s.fr (Postfix) with ESMTP id 2E6BF8B8F9; Wed, 15 May 2019 11:26:08 +0200 (CEST) X-Virus-Scanned: amavisd-new at c-s.fr Received: from messagerie.si.c-s.fr ([127.0.0.1]) by localhost (messagerie.si.c-s.fr [127.0.0.1]) (amavisd-new, port 10023) with ESMTP id RCvOx4aCNG0I; Wed, 15 May 2019 11:26:08 +0200 (CEST) Received: from PO15451 (po15451.idsi0.si.c-s.fr [172.25.231.1]) by messagerie.si.c-s.fr (Postfix) with ESMTP id DD35F8B8F7; Wed, 15 May 2019 11:26:07 +0200 (CEST) Subject: Re: [PATCH] powerpc: Remove double free To: "Tobin C. Harding" , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman Cc: Greg Kroah-Hartman , linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org References: <20190515090750.30647-1-tobin@kernel.org> From: Christophe Leroy Message-ID: Date: Wed, 15 May 2019 11:26:03 +0200 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1 MIME-Version: 1.0 In-Reply-To: <20190515090750.30647-1-tobin@kernel.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: fr Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org kobject_put() released index_dir->kobj but who will release 'index' ? Christophe Le 15/05/2019 à 11:07, Tobin C. Harding a écrit : > kfree() after kobject_put(). Who ever wrote this was on crack. > > Fixes: 7e8039795a80 ("powerpc/cacheinfo: Fix kobject memleak") > Signed-off-by: Tobin C. Harding > --- > > FTR > > git log --pretty=format:"%h%x09%an%x09%ad%x09%s" | grep 7e8039795a80 > 7e8039795a80 Tobin C. Harding Tue Apr 30 11:09:23 2019 +1000 powerpc/cacheinfo: Fix kobject memleak > > arch/powerpc/kernel/cacheinfo.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/powerpc/kernel/cacheinfo.c b/arch/powerpc/kernel/cacheinfo.c > index f2ed3ef4b129..862e2890bd3d 100644 > --- a/arch/powerpc/kernel/cacheinfo.c > +++ b/arch/powerpc/kernel/cacheinfo.c > @@ -767,7 +767,6 @@ static void cacheinfo_create_index_dir(struct cache *cache, int index, > cache_dir->kobj, "index%d", index); > if (rc) { > kobject_put(&index_dir->kobj); > - kfree(index_dir); > return; > } > >