lkml.org 
[lkml]   [2020]   [Nov]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 2/5] clk: Add CLK_GET_PARENT_NOCACHE flag (fwd)
Please check line 2432.  Based on the preceeding tests, there may be a
NULL pointer dereference.

julia

---------- Forwarded message ----------
Date: Fri, 27 Nov 2020 06:27:51 +0800
From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Cc: lkp@intel.com, Julia Lawall <julia.lawall@lip6.fr>
Subject: Re: [PATCH v2 2/5] clk: Add CLK_GET_PARENT_NOCACHE flag

CC: kbuild-all@lists.01.org
In-Reply-To: <1606394409-12755-3-git-send-email-abel.vesa@nxp.com>
References: <1606394409-12755-3-git-send-email-abel.vesa@nxp.com>
TO: Abel Vesa <abel.vesa@nxp.com>
TO: Stephen Boyd <sboyd@kernel.org>
TO: Sascha Hauer <kernel@pengutronix.de>
TO: Peng Fan <peng.fan@nxp.com>
TO: Fabio Estevam <fabio.estevam@nxp.com>
TO: Anson Huang <anson.huang@nxp.com>
TO: Dong Aisheng <aisheng.dong@nxp.com>
TO: Jacky Bai <ping.bai@nxp.com>
CC: NXP Linux Team <linux-imx@nxp.com>
CC: linux-clk@vger.kernel.org
CC: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>

Hi Abel,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on shawnguo/for-next]
[also build test WARNING on clk/clk-next v5.10-rc5 next-20201126]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url: https://github.com/0day-ci/linux/commits/Abel-Vesa/clk-imx-Register-the-dram_apb-and-dram_alt-as-read-only/20201126-204442
base: https://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git for-next
:::::: branch date: 10 hours ago
:::::: commit date: 10 hours ago
config: x86_64-randconfig-c002-20201127 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Julia Lawall <julia.lawall@lip6.fr>


"coccinelle warnings: (new ones prefixed by >>)"
>> drivers/clk/clk.c:2432:23-29: ERROR: clk -> core is NULL but dereferenced.
drivers/clk/clk.c:2432:50-56: ERROR: clk -> core is NULL but dereferenced.

vim +2432 drivers/clk/clk.c

05e4e881cde17e7 Abel Vesa 2020-11-26 2413
4dff95dc9477a34 Stephen Boyd 2015-04-30 2414 /**
4dff95dc9477a34 Stephen Boyd 2015-04-30 2415 * clk_get_parent - return the parent of a clk
4dff95dc9477a34 Stephen Boyd 2015-04-30 2416 * @clk: the clk whose parent gets returned
4dff95dc9477a34 Stephen Boyd 2015-04-30 2417 *
4dff95dc9477a34 Stephen Boyd 2015-04-30 2418 * Simply returns clk->parent. Returns NULL if clk is NULL.
4935b22c46ea5e2 James Hogan 2013-07-29 2419 */
4dff95dc9477a34 Stephen Boyd 2015-04-30 2420 struct clk *clk_get_parent(struct clk *clk)
4dff95dc9477a34 Stephen Boyd 2015-04-30 2421 {
4dff95dc9477a34 Stephen Boyd 2015-04-30 2422 struct clk *parent;
da0f0b2c3ad2ad9 Tomasz Figa 2013-09-29 2423
fc4a05d4b0eb1a0 Stephen Boyd 2015-06-25 2424 if (!clk)
fc4a05d4b0eb1a0 Stephen Boyd 2015-06-25 2425 return NULL;
fc4a05d4b0eb1a0 Stephen Boyd 2015-06-25 2426
4dff95dc9477a34 Stephen Boyd 2015-04-30 2427 clk_prepare_lock();
05e4e881cde17e7 Abel Vesa 2020-11-26 2428 if (clk->core && (clk->core->flags & CLK_GET_PARENT_NOCACHE))
05e4e881cde17e7 Abel Vesa 2020-11-26 2429 parent = __clk_get_parent(clk->core)->hw->clk;
05e4e881cde17e7 Abel Vesa 2020-11-26 2430 else
fc4a05d4b0eb1a0 Stephen Boyd 2015-06-25 2431 /* TODO: Create a per-user clk and change callers to call clk_put */
fc4a05d4b0eb1a0 Stephen Boyd 2015-06-25 @2432 parent = !clk->core->parent ? NULL : clk->core->parent->hw->clk;
4dff95dc9477a34 Stephen Boyd 2015-04-30 2433 clk_prepare_unlock();
4935b22c46ea5e2 James Hogan 2013-07-29 2434
4dff95dc9477a34 Stephen Boyd 2015-04-30 2435 return parent;
4935b22c46ea5e2 James Hogan 2013-07-29 2436 }
4dff95dc9477a34 Stephen Boyd 2015-04-30 2437 EXPORT_SYMBOL_GPL(clk_get_parent);
4935b22c46ea5e2 James Hogan 2013-07-29 2438

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org[unhandled content-type:application/gzip]
\
 
 \ /
  Last update: 2020-11-27 14:31    [W:0.037 / U:0.336 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site