Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp5475110yba; Tue, 30 Apr 2019 15:42:56 -0700 (PDT) X-Google-Smtp-Source: APXvYqxZZpE2IbmiWzjMxFjBEEAKLjQlsBz48q7Npf+ZdHx7Exeg3jQb3EJkltlKkOI2SXuDLwVD X-Received: by 2002:a63:5d46:: with SMTP id o6mr34783872pgm.217.1556664176724; Tue, 30 Apr 2019 15:42:56 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1556664176; cv=none; d=google.com; s=arc-20160816; b=0mxSx9nLodQEsAgTIKIJ35a5TkqgummUVUM5VsQA53XoiLYo7VZcPig1igChXAJNu0 xjr0nkTRv+kJzHqfyjLqDbnFJKNJ65DIHznQ4ZHbv64h6kuTYSymt3DfvJiy1tIe6eox tZZ/1TBBAdMYhHcA3Raf8OdQZEyakaIf9CgklLnicgGvDynAbFIyw1WSAc7Z9slKJesv +Z2z1KM9Rsy4zUIchrTND8ecxFCsTABbxmZo3vpQDNKSPhVIq34kzIbwynztLRp4iABp ot2xGDsL0NHTJBWmCS60h/f0IcqlvXG8OT8kfCNhtbLxFmxr4daXKuvKbGT97j3VXUoB 5UJw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature:dkim-signature; bh=uSxjqNftJ9OpafDOpoXJOCCkS+qPtA+kvIr1Yk5mjnU=; b=rFbklLMrJJDO8RNJS4vq54xOphmYkJXKkIbsiWV8v5idSNelg5hFfmIc7hWyzZBwLq msev9osWR0CbYpN+jxdYzqCgNSx63guL5YpTDnc1qFgl1AjVHmSFddJCad2aJ+JIRn+r MRSwH54Bl/a74gmyNC23hq1yTdCtipxDnjoKon3nXH+zQLfBHVTKLJkFEgajHbtnhzPL PvNNaDwm7uTjsqOd6M0Nu3W3qzks0XDv2tT+0WInW1wtIW3PtU6/uv+zmuusVG0WWS6y WyHFp0li2U13wpR6WJUcxhs7fBSZHNGGsgCP/fOmV1If4kgI4c3xusOImuqWMBEraZbF PxWg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@tobin.cc header.s=fm3 header.b=OVblrJdI; dkim=pass header.i=@messagingengine.com header.s=fm2 header.b=U8r20nCq; 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 x188si21270725pfx.76.2019.04.30.15.42.40; Tue, 30 Apr 2019 15:42:56 -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=@tobin.cc header.s=fm3 header.b=OVblrJdI; dkim=pass header.i=@messagingengine.com header.s=fm2 header.b=U8r20nCq; 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 S1727388AbfD3Wkg (ORCPT + 99 others); Tue, 30 Apr 2019 18:40:36 -0400 Received: from out2-smtp.messagingengine.com ([66.111.4.26]:46365 "EHLO out2-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726056AbfD3Wkf (ORCPT ); Tue, 30 Apr 2019 18:40:35 -0400 Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id B97612329E; Tue, 30 Apr 2019 18:40:34 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute5.internal (MEProxy); Tue, 30 Apr 2019 18:40:34 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tobin.cc; h=date :from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm3; bh=uSxjqNftJ9OpafDOpoXJOCCkS+q PtA+kvIr1Yk5mjnU=; b=OVblrJdIh9NfZAks9MSGdeTlodHm49AyY2o11wxi710 n3/+hM8UM0DLZjBqBfvIflQWqVD1brVvxGCdQMMg5CG83sRX3OTubzXPilqhPljg fJDhQXHVRMwe0rWKcx+Bd0rBEB/oGGxlZNdH6hxvYGlO9bxsqarIuFYeCy+rd4em yYbPWXTr9ih200bXgEQdD4zOMROiTk/N2NclJd2Vmw4DDVzdRKyd+czThEUG5TZE sfD1tDEiepc4VsIf1YbP8Dx92YbyJofywbpC9TE1gtA0I0aI/nsLZo8alQBXoiXv OGgSoYzkaaj+dw3vWpV1YHfUVtlPDYm7IPKXf0U55lg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; bh=uSxjqN ftJ9OpafDOpoXJOCCkS+qPtA+kvIr1Yk5mjnU=; b=U8r20nCq5Ym2C6mmddce/E nK7Js5/U+3l4k/+6ficF1l3c42PsXmZbVoRUrbJQuZjWrz/Fv5qgDBcJI2rXfI98 pWH/10DOwpXtPV3N8626w9a3ULwoI9jfmeETK9J642xAMGiIIs6eSrgQpV4LTlFQ gezJquUfpMdu9fPcBG+N3hSpZKr45BWgc/Sost6CuoEVOTQOr8BQnpDjMX960Tnm ypGVN9QxUMuz3eSqSRXqyKJEJJOBoWHyuCc8maH5/YNnWMlu6iP768e05pabKPSz Ax/N6lfrV6VaT5CrgXcA5EBIfhSaWFRMNSnryP3qVOc1o/G3hLknC6T6efibbycw == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduuddrieeigddufecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenfg hrlhcuvffnffculdduhedmnecujfgurhepfffhvffukfhfgggtuggjofgfsehttdertdfo redvnecuhfhrohhmpedfvfhosghinhcuvedrucfjrghrughinhhgfdcuoehmvgesthhosg hinhdrtggtqeenucfkphepuddvuddrgeegrddvtdegrddvfeehnecurfgrrhgrmhepmhgr ihhlfhhrohhmpehmvgesthhosghinhdrtggtnecuvehluhhsthgvrhfuihiivgepud X-ME-Proxy: Received: from localhost (ppp121-44-204-235.bras1.syd2.internode.on.net [121.44.204.235]) by mail.messagingengine.com (Postfix) with ESMTPA id 0A6E8103CA; Tue, 30 Apr 2019 18:40:32 -0400 (EDT) Date: Wed, 1 May 2019 08:39:57 +1000 From: "Tobin C. Harding" To: Miroslav Benes Cc: "Tobin C. Harding" , Greg Kroah-Hartman , Josh Poimboeuf , Jiri Kosina , Petr Mladek , Joe Lawrence , live-patching@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] livepatch: Fix kobject memleak Message-ID: <20190430223957.GF9454@eros.localdomain> References: <20190430001534.26246-1-tobin@kernel.org> <20190430001534.26246-2-tobin@kernel.org> <20190430084254.GB11737@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Mailer: Mutt 1.11.4 (2019-03-13) User-Agent: Mutt/1.11.4 (2019-03-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Apr 30, 2019 at 12:44:55PM +0200, Miroslav Benes wrote: > On Tue, 30 Apr 2019, Greg Kroah-Hartman wrote: > > > On Tue, Apr 30, 2019 at 10:15:33AM +1000, Tobin C. Harding wrote: > > > Currently error return from kobject_init_and_add() is not followed by a > > > call to kobject_put(). This means there is a memory leak. > > > > > > Add call to kobject_put() in error path of kobject_init_and_add(). > > > > > > Signed-off-by: Tobin C. Harding > > > > Reviewed-by: Greg Kroah-Hartman > > Well, it does not even compile... My apologies, I did compile this but obviously I don't know how to configure the kernel. Thanks for the review. Tobin