Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp4291054yba; Mon, 29 Apr 2019 17:30:33 -0700 (PDT) X-Google-Smtp-Source: APXvYqzFTkwy30PfxfnIM7rUD5J9Gx3dB810MMNBHcXzHqpCSkduByv1fSmVu/2mNTxeYQRjeGgC X-Received: by 2002:a17:902:5a03:: with SMTP id q3mr10406001pli.209.1556584233582; Mon, 29 Apr 2019 17:30:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1556584233; cv=none; d=google.com; s=arc-20160816; b=MGLk9yaYrCB83V5cHdYPuhhVNTKpcl7OxyVcLmJHj2aTOD6dimKSuIJMfF+JBwNy9O aaskBICUj5Y7e0gGTZ/43bpaOcrZUYX4szgtmXa+1u5TPKMLcUwHC5c0G+VjikL7Tgdy ItyF2uLxH3llSZVzCl+pqjNs8Zki6UarnqXT2TFBRzEeklkH7kKTyGnI0QZUlQ9jxjae bSb+j9btaXnvEQyW784umjWP8tlgTRBCWylDMxWVXNlqfz9XjZxVEJsqsCaNRP9SZx2z 5Okef4s2sgZUnX0shQ+JniAsyj7VDk8+0qo8usFqe/0aDh0l4LBOG21fFtlc8caMtQH1 VG4Q== 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:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=2xZH67dVb6wgUB263feKLZ/OFq7jqKpOxkOzPVJS6gM=; b=eGGdQSrGZnlwT22LBkiJdfHTwPVIW74NZJW3SvVWENR2guHoC4+uXfOji9slFbErFL 76R0h2cNqltR4V+z/6ysHRE8LeLcGja8eTevn37LfXrJn62FYxeOMvsbDORU7utSbDl8 YvbQXiXbJq1Yc2TKZVsQfosYOai7dq663BRSPTwGN8bKMUQ5CKU4Y2I0RxQE96C1ukBz DFEvOj6gINtcCUejFPc1f0gav6p0YfZ+ygbDrR12FqS++WDl3Z8w95m8UoEctP1ql5cB 04i3LoXA6FCtkqLYL6wiZ1JH4U0AiNfEpfazONRsoEacNDY7k1BRHml2Mxmfmt9UsMTv WdZw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@messagingengine.com header.s=fm2 header.b=VObclZG+; 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s189si19560739pgb.346.2019.04.29.17.30.17; Mon, 29 Apr 2019 17:30:33 -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=@messagingengine.com header.s=fm2 header.b=VObclZG+; 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729739AbfD3A3R (ORCPT + 99 others); Mon, 29 Apr 2019 20:29:17 -0400 Received: from new1-smtp.messagingengine.com ([66.111.4.221]:43281 "EHLO new1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729214AbfD3A3Q (ORCPT ); Mon, 29 Apr 2019 20:29:16 -0400 Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailnew.nyi.internal (Postfix) with ESMTP id 736234113; Mon, 29 Apr 2019 20:29:14 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute3.internal (MEProxy); Mon, 29 Apr 2019 20:29:14 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding: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=2xZH67dVb6wgUB263feKLZ/OFq7jqKpOxkOzPVJS6gM=; b=VObclZG+ JIqqX9GoACkcqVWTp951RK98sL8sRjfzTNQSMOf3W91Cq8K7ZZXRShgfskt0E8Mu rskBPDJzjDkI+lZ/qzbmflZDSDntNQsUfVgbuCPfSxK0fnv/f8xR/m3PEC1cUdpk 8Eqa9Fk8+VUZpOSRGh8iGZp4U4cLCxwiDEumNarOyx01Ygu6jGj4q7s+Jk1FR9o1 D4RDczHOQd01ZvSMmq+OKV4YnFyyBuTzr+YVx6OSIPuFtpzCwHA7FgqrCCgLDG8z XMyamE0tMNs92bWnkWkeoKV5Dzt5m3iJdmGRvkuDM4Qnn7IY7cLleFZ3X80+u0Bb CBfnUmBKy3lylQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduuddrieefgdefhecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefhvffufffkofgjfhgggfestdekredtredttdenucfhrhhomhepfdfvohgsihhn ucevrdcujfgrrhguihhnghdfuceothhosghinheskhgvrhhnvghlrdhorhhgqeenucfkph epuddvuddrgeegrddvfedtrddukeeknecurfgrrhgrmhepmhgrihhlfhhrohhmpehtohgs ihhnsehkvghrnhgvlhdrohhrghenucevlhhushhtvghrufhiiigvpedt X-ME-Proxy: Received: from eros.localdomain (ppp121-44-230-188.bras2.syd2.internode.on.net [121.44.230.188]) by mail.messagingengine.com (Postfix) with ESMTPA id A5001103CA; Mon, 29 Apr 2019 20:29:09 -0400 (EDT) From: "Tobin C. Harding" To: "David S. Miller" Cc: "Tobin C. Harding" , Greg Kroah-Hartman , Tyler Hicks , Andy Shevchenko , Ido Schimmel , Alexander Duyck , Florian Fainelli , Wang Hai , YueHaibing , Amritha Nambiar , Dmitry Torokhov , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH 2/3] bridge: Use correct cleanup function Date: Tue, 30 Apr 2019 10:28:16 +1000 Message-Id: <20190430002817.10785-3-tobin@kernel.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190430002817.10785-1-tobin@kernel.org> References: <20190430002817.10785-1-tobin@kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The correct cleanup function if a call to kobject_init_and_add() has returned _successfully_ is kobject_del(). kobject_put() is used if the call to kobject_init_and_add() fails. kobject_del() calls kobject_put(). Use correct cleanup function in error path. Signed-off-by: Tobin C. Harding --- net/bridge/br_if.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c index e5c8c9941c51..d3a1554ccff4 100644 --- a/net/bridge/br_if.c +++ b/net/bridge/br_if.c @@ -701,7 +701,7 @@ int br_add_if(struct net_bridge *br, struct net_device *dev, err3: sysfs_remove_link(br->ifobj, p->dev->name); err2: - kobject_put(&p->kobj); + kobject_del(&p->kobj); p = NULL; /* kobject_put frees */ err1: dev_set_allmulti(dev, -1); -- 2.21.0