Received: by 10.223.185.116 with SMTP id b49csp2146293wrg; Thu, 15 Feb 2018 07:15:20 -0800 (PST) X-Google-Smtp-Source: AH8x226C7LB54gq5aOlcV3kQkBaQvPVY6sMB3uiPuFgAZfm986G+iBv/++FrjQN9U+VvF2fFQx2T X-Received: by 10.99.157.193 with SMTP id i184mr2430595pgd.197.1518707720534; Thu, 15 Feb 2018 07:15:20 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1518707720; cv=none; d=google.com; s=arc-20160816; b=ydawuO7CBSGGCclh9AHltEHrg7z8CNGgrzaSijJKaq3t056uDM0BQPWLFwmQYrWP70 naZxhHbrNSZE2hpRaCVwp84YrCaoniYm2CosoSQarOeD7EZKKlt5dI8DqnMPmsEiCvCm aQl/eu6OyXxa2wHG0kUY6M0BDaasrZUVx/9KgrawO3r9ZD+RyhzVAqFPOOywOTGulszS IBV0O35BtpJ5jfaznIPCUoZSz3IJ75Wuz9p3x+T+4xydZLCiy4u08YFy1kVmBF+U7mkm 5FxRr4pWLwGHLLcLbjJ9CsQuHueBLfbEsS76X7O7oXR/S4GusfLRvq+jF7PCFlo8ifg9 oNOw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:content-disposition :mime-version:message-id:subject:cc:to:from:date :arc-authentication-results; bh=mxIiT1og0qiQgjHGIgGrzuFxP04FhBISdeN7pS2NY10=; b=xNA4CtS4zae0SKxJhZbl3evxy4yvk3yq+utH5GKG7QdF3kD2ObU60eeroZym7PrfpU zzfvaDuEiPBo7LaawzeSI/dPmrYSETypElKLi6WZoR+YKmI/5axDdKJSmwsn5CvIXazj ieIzsu8ODE9J23TZLTRczYoJ4CuD0nZ9Mr6hK1zSCVfiMxsBwtYlz5dh2Nu0/lHhW9lY kV/m6YwBZZld2dM0NLB0E7e+hMCmL6epl4KosL2mUpSCEeJUNabD9tN3+Idl69EyE+no EjLeoGSEf1RViivbMw5AE+ukFxPw95xjJYfH4tunm+9gDFOD6p5Pg0btZDZbAHzPFEMm 8KaQ== ARC-Authentication-Results: i=1; mx.google.com; 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 p9-v6si1660522plk.6.2018.02.15.07.15.05; Thu, 15 Feb 2018 07:15:20 -0800 (PST) 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; 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 S1033786AbeBOPM7 (ORCPT + 99 others); Thu, 15 Feb 2018 10:12:59 -0500 Received: from agrajag.zerfleddert.de ([88.198.237.222]:40860 "EHLO agrajag.zerfleddert.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1033584AbeBOPM5 (ORCPT ); Thu, 15 Feb 2018 10:12:57 -0500 Received: by agrajag.zerfleddert.de (Postfix, from userid 1000) id 9BBD65B2019D; Thu, 15 Feb 2018 16:12:55 +0100 (CET) Date: Thu, 15 Feb 2018 16:12:55 +0100 From: Tobias Jordan To: Ohad Ben-Cohen , Bjorn Andersson , linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org Cc: sil2reviews@lists.osadl.org Subject: [PATCH] remoteproc: qcom: Fix potential device node leaks Message-ID: <20180215151255.xhe7gzxnwippv3gp@agrajag.zerfleddert.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Add missing of_node_put()s at two places for device nodes returned by of_parse_phandle(). Fixes: 051fb70fd4ea ("remoteproc: qcom: Driver for the self-authenticating Hexagon v5") Signed-off-by: Tobias Jordan --- drivers/remoteproc/qcom_q6v5_pil.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/remoteproc/qcom_q6v5_pil.c b/drivers/remoteproc/qcom_q6v5_pil.c index 8e70a627e0bb..cbbafdcaaecb 100644 --- a/drivers/remoteproc/qcom_q6v5_pil.c +++ b/drivers/remoteproc/qcom_q6v5_pil.c @@ -1083,6 +1083,7 @@ static int q6v5_alloc_memory_region(struct q6v5 *qproc) dev_err(qproc->dev, "unable to resolve mba region\n"); return ret; } + of_node_put(node); qproc->mba_phys = r.start; qproc->mba_size = resource_size(&r); @@ -1100,6 +1101,7 @@ static int q6v5_alloc_memory_region(struct q6v5 *qproc) dev_err(qproc->dev, "unable to resolve mpss region\n"); return ret; } + of_node_put(node); qproc->mpss_phys = qproc->mpss_reloc = r.start; qproc->mpss_size = resource_size(&r); -- 2.11.0