From patchwork Fri Mar 21 12:18:17 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Krzysztof Kozlowski X-Patchwork-Id: 3873511 Return-Path: X-Original-To: patchwork-linux-samsung-soc@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 643BFBF540 for ; Fri, 21 Mar 2014 12:18:34 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 937AF2026C for ; Fri, 21 Mar 2014 12:18:33 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6155520268 for ; Fri, 21 Mar 2014 12:18:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760098AbaCUMSb (ORCPT ); Fri, 21 Mar 2014 08:18:31 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:52546 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755604AbaCUMSa (ORCPT ); Fri, 21 Mar 2014 08:18:30 -0400 Received: from eucpsbgm2.samsung.com (unknown [203.254.199.245]) by mailout4.w1.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0N2S00DI1CURMN50@mailout4.w1.samsung.com>; Fri, 21 Mar 2014 12:18:27 +0000 (GMT) X-AuditID: cbfec7f5-b7fc96d000004885-c6-532c2e14a54c Received: from eusync4.samsung.com ( [203.254.199.214]) by eucpsbgm2.samsung.com (EUCPMTA) with SMTP id 8A.9A.18565.41E2C235; Fri, 21 Mar 2014 12:18:28 +0000 (GMT) Received: from AMDC1943.digital.local ([106.116.151.171]) by eusync4.samsung.com (Oracle Communications Messaging Server 7u4-24.01(7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPA id <0N2S00F4DCUOI000@eusync4.samsung.com>; Fri, 21 Mar 2014 12:18:27 +0000 (GMT) From: Krzysztof Kozlowski To: Mike Turquette , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org Cc: Tushar Behera , Kyungmin Park , Marek Szyprowski , Bartlomiej Zolnierkiewicz , Tomasz Figa , Krzysztof Kozlowski , stable@vger.kernel.org Subject: [PATCH 1/2] clk: s2mps11: Fix possible NULL pointer dereference Date: Fri, 21 Mar 2014 13:18:17 +0100 Message-id: <1395404297-12133-1-git-send-email-k.kozlowski@samsung.com> X-Mailer: git-send-email 1.7.9.5 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrNJMWRmVeSWpSXmKPExsVy+t/xa7oiejrBBou/M1psnLGe1eL1C0OL s01v2C02Pb7GanF51xw2ixnn9zFZrD1yl93i6YSLbBYLNj5itFg/4zWLRfvfvWwO3B53ru1h 89i8pN6jb8sqRo/Pm+QCWKK4bFJSczLLUov07RK4Mu69PcxWcJCtomXdO9YGxr2sXYycHBIC JhIn3/VD2WISF+6tZ+ti5OIQEljKKHH+2BIop49JYuLDW4wgVWwCxhKbl0MkRASmMkpsXL+I CcRhFljLJLF/2RaWLkYODmEBD4nN99NAGlgEVCUmTV7ADmLzCrhL9P7YDVYiIaAgMWeSzQRG 7gWMDKsYRVNLkwuKk9JzjfSKE3OLS/PS9ZLzczcxQgLo6w7GpcesDjEKcDAq8fBWcGoHC7Em lhVX5h5ilOBgVhLhldPSCRbiTUmsrEotyo8vKs1JLT7EyMTBKdXAWP0vq3RdlUNN+96TsmGb uUxXf/i8oPLtlZSP7VpKhpzs1p/2eVmX7sy5GuNgXjv758ETd2t41arOifcvT2RcXfDlzU6r 8C0nqjy9fnl89v7zMVW2aM0nY5ebidkGK/3vLpp2t/mqiqivzlonr4X7OLaG72aRsV/nVvF4 0f7V/BZ7X31+oDfJRomlOCPRUIu5qDgRABVTA/H+AQAA Sender: linux-samsung-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP If parent device does not have of_node set the s2mps11_clk_parse_dt() returned NULL. This NULL was later passed to of_clk_add_provider() which dereferenced it in pr_debug() call. Signed-off-by: Krzysztof Kozlowski Cc: --- drivers/clk/clk-s2mps11.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/clk-s2mps11.c b/drivers/clk/clk-s2mps11.c index 00a3abe103a5..27c83e45eaed 100644 --- a/drivers/clk/clk-s2mps11.c +++ b/drivers/clk/clk-s2mps11.c @@ -130,7 +130,7 @@ static struct device_node *s2mps11_clk_parse_dt(struct platform_device *pdev) int i; if (!iodev->dev->of_node) - return NULL; + return ERR_PTR(-EINVAL); clk_np = of_find_node_by_name(iodev->dev->of_node, "clocks"); if (!clk_np) {